diff options
| author | Jeff King <peff@peff.net> | 2022-11-28 22:09:52 +0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-11-29 09:54:56 +0900 |
| commit | 833f4c0514fb08368066ace5260b4b80792c4ffc (patch) | |
| tree | c9ac87d420a86327145f2e21497d7f915c0da327 | |
| parent | pack-bitmap.c: avoid exposing absolute paths (diff) | |
| download | git-833f4c0514fb08368066ace5260b4b80792c4ffc.tar.gz git-833f4c0514fb08368066ace5260b4b80792c4ffc.zip | |
pack-bitmap.c: break out of the bitmap loop early if not tracing
After opening a bitmap successfully, we try opening others only
because we want to report that other bitmap files are ignored in
the trace2 log. When trace2 is not enabled, we do not have to
do any of that.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Teng Long <dyroneteng@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | pack-bitmap.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/pack-bitmap.c b/pack-bitmap.c index aaa2d9a104..3b6c2f804a 100644 --- a/pack-bitmap.c +++ b/pack-bitmap.c @@ -527,8 +527,15 @@ static int open_pack_bitmap(struct repository *r, assert(!bitmap_git->map); for (p = get_all_packs(r); p; p = p->next) { - if (open_pack_bitmap_1(bitmap_git, p) == 0) + if (open_pack_bitmap_1(bitmap_git, p) == 0) { ret = 0; + /* + * The only reason to keep looking is to report + * duplicates. + */ + if (!trace2_is_enabled()) + break; + } } return ret; |