From: Emily Shaffer <emilyshaffer@google.com>
Date: Fri, 22 Oct 2021 03:37:35 GMT
Subject: Re: [PATCH v3 08/10] pack-bitmap-write.c: don't return without stop_progress()
Message-ID: <YXIx/8Hl+BG7HFV9@google.com>
In-Reply-To: <patch-v3-08.10-e0a3510dd88-20211013T222329Z-avarab@gmail.com>

On Thu, Oct 14, 2021 at 12:28:24AM +0200, Ævar Arnfjörð Bjarmason wrote:
> 
> Fix a bug that's been here since 7cc8f971085 (pack-objects: implement
> bitmap writing, 2013-12-21), we did not call stop_progress() if we
> reached the early exit in this function.
> 
> We could call stop_progress() before we return, but better yet is to
> defer calling start_progress() until we need it.
> 
> This will matter in a subsequent commit where we BUG(...) out if this
> happens, and matters now e.g. because we don't have a corresponding
> "region_end" for the progress trace2 event.
> 
> Suggested-by: SZEDER Gábor <szeder.dev@gmail.com>
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Very nice redo.
Reviewed-by: Emily Shaffer <emilyshaffer@google.com>