diff options
| author | Derrick Stolee <derrickstolee@github.com> | 2022-03-09 16:01:36 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-03-09 10:25:27 -0800 |
| commit | 3e0370a8d2251742d583ce095072b7dcc34b3c01 (patch) | |
| tree | 4d5e7d29c452261687da3a1bcae66fb35fa2b338 /builtin/pack-objects.c | |
| parent | pack-bitmap: drop filter in prepare_bitmap_walk() (diff) | |
| download | git-3e0370a8d2251742d583ce095072b7dcc34b3c01.tar.gz git-3e0370a8d2251742d583ce095072b7dcc34b3c01.zip | |
list-objects: consolidate traverse_commit_list[_filtered]
Now that all consumers of traverse_commit_list_filtered() populate the
'filter' member of 'struct rev_info', we can drop that parameter from
the method prototype to simplify things. In addition, the only thing
different now between traverse_commit_list_filtered() and
traverse_commit_list() is the presence of the 'omitted' parameter, which
is only non-NULL for one caller. We can consolidate these two methods by
having one call the other and use the simpler form everywhere the
'omitted' parameter would be NULL.
Signed-off-by: Derrick Stolee <derrickstolee@github.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/pack-objects.c')
| -rw-r--r-- | builtin/pack-objects.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c index ddd698a1fe..ae7dbc8c8a 100644 --- a/builtin/pack-objects.c +++ b/builtin/pack-objects.c @@ -3778,9 +3778,9 @@ static void get_object_list(int ac, const char **av) if (!fn_show_object) fn_show_object = show_object; - traverse_commit_list_filtered(&revs.filter, &revs, - show_commit, fn_show_object, NULL, - NULL); + traverse_commit_list(&revs, + show_commit, fn_show_object, + NULL); if (unpack_unreachable_expiration) { revs.ignore_missing_links = 1; |