diff options
| author |  Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-02-03 22:40:16 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2022-02-03 15:39:55 -0800 |
| commit | 1ccad6a1f175080c3896a70501dcd6c9e0a0af0a (patch) | |
| tree | daac109caa93bf9994dbffbd2e73d7816034a688 /t/t4100 | |
| parent | progress.h: format and be consistent with progress.c naming (diff) | |
| download | tgif-1ccad6a1f175080c3896a70501dcd6c9e0a0af0a.tar.xz | |
progress.c: use dereferenced "progress" variable, not "(*p_progress)"
Since 98a13647408 (trace2: log progress time and throughput,
2020-05-12) stop_progress() dereferences a "struct progress **"
parameter in several places. Extract a dereferenced variable to reduce
clutter and make it clearer who needs to write to this parameter.
Now instead of using "*p_progress" several times in stop_progress() we
check it once for NULL and then use a dereferenced "progress" variable
thereafter. This uses the same pattern as the adjacent
stop_progress_msg() function, see ac900fddb7f (progress: don't
dereference before checking for NULL, 2020-08-10).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4100')
0 files changed, 0 insertions, 0 deletions