diff options
| author | René Scharfe <l.s.r@web.de> | 2022-10-04 18:17:39 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-10-05 21:46:27 +0200 |
| commit | b004c9028270832aec73746fdddd5c7bce8dbe84 (patch) | |
| tree | 23e4976cec6d26f3d4551a27ce234634dd0880e4 /builtin/gc.c | |
| parent | Git 2.38 (diff) | |
| download | git-b004c9028270832aec73746fdddd5c7bce8dbe84.tar.xz git-b004c9028270832aec73746fdddd5c7bce8dbe84.zip | |
gc: simplify maintenance_task_pack_refs()
Pass a constant string array directly to run_command_v_opt() instead of
copying it into a strvec first. This shortens the code and avoids heap
allocations.
Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to '')
| -rw-r--r-- | builtin/gc.c | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/builtin/gc.c b/builtin/gc.c index 2753bd15a5..ceff31ea00 100644 --- a/builtin/gc.c +++ b/builtin/gc.c @@ -167,16 +167,9 @@ static void gc_config(void) struct maintenance_run_opts; static int maintenance_task_pack_refs(MAYBE_UNUSED struct maintenance_run_opts *opts) { - struct strvec pack_refs_cmd = STRVEC_INIT; - int ret; + const char *argv[] = { "pack-refs", "--all", "--prune", NULL }; - strvec_pushl(&pack_refs_cmd, "pack-refs", "--all", "--prune", NULL); - - ret = run_command_v_opt(pack_refs_cmd.v, RUN_GIT_CMD); - - strvec_clear(&pack_refs_cmd); - - return ret; + return run_command_v_opt(argv, RUN_GIT_CMD); } static int too_many_loose_objects(void) |