Subject: Re: [PATCH v2 6/8] progress.c: add temporary variable from progress struct

From: Emily Shaffer <[email-masked]>
Date: Thu, 7 Oct 2021 21:02:37 -0700

On Tue, Sep 21, 2021 at 01:09:27AM +0200, Ævar Arnfjörð Bjarmason wrote:
> 
> Add a temporary "progress" variable for the dereferenced p_progress
> pointer to a "struct progress *". Before 98a13647408 (trace2: log
> progress time and throughput, 2020-05-12) we didn't dereference
> "p_progress" in this function, now that we do it's easier to read the
> code if we work with a "progress" struct pointer like everywhere else,
> instead of a pointer to a pointer.

Thanks, this looks much nicer :)

> 
> Signed-off-by: Ævar Arnfjörð Bjarmason <[email-masked]>

Reviewed-by: Emily Shaffer <[email-masked]>