relax usage of the progress API
Since it is now OK to pass a null pointer to display_progress() and stop_progress() resulting in a no-op, then we can simplify the code and remove a bunch of lines by not making those calls conditional all the time. Signed-off-by: Nicolas Pitre <nico@cam.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
committed by
Junio C Hamano
parent
dc6a0757c4
commit
4d4fcc5451
@ -316,9 +316,8 @@ static void check_updates(struct cache_entry **src, int nr,
|
||||
while (nr--) {
|
||||
struct cache_entry *ce = *src++;
|
||||
|
||||
if (total)
|
||||
if (!ce->ce_mode || ce->ce_flags & mask)
|
||||
display_progress(progress, ++cnt);
|
||||
if (!ce->ce_mode || ce->ce_flags & mask)
|
||||
display_progress(progress, ++cnt);
|
||||
if (!ce->ce_mode) {
|
||||
if (o->update)
|
||||
unlink_entry(ce->name, last_symlink);
|
||||
@ -332,8 +331,7 @@ static void check_updates(struct cache_entry **src, int nr,
|
||||
}
|
||||
}
|
||||
}
|
||||
if (total)
|
||||
stop_progress(&progress);
|
||||
stop_progress(&progress);
|
||||
}
|
||||
|
||||
int unpack_trees(unsigned len, struct tree_desc *t, struct unpack_trees_options *o)
|
||||
|
||||
Reference in New Issue
Block a user