[PATCH v2 5/8] builtin/repack.c: extract showing progress to a variable
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Sep 15, 2021, 18:24 UTC
- Message-ID
- <09de03de475d9bd38fa1ce500c45df6c40694cbf.1631730270.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1631730270.git.me@ttaylorr.com>
We only ask whether stderr is a tty before calling 'prune_packed_objects()', but the subsequent patch will add another use.
Extract this check into a variable so that both can use it without having to call 'isatty()' twice.
Signed-off-by: Taylor Blau <me@ttaylorr.com> --- builtin/repack.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/builtin/repack.c b/builtin/repack.c index e55a650de5..be470546e6 100644 --- a/builtin/repack.c +++ b/builtin/repack.c @@ -445,6 +445,7 @@ int cmd_repack(int argc, const char **argv, const char *prefix) struct strbuf line = STRBUF_INIT; int i, ext, ret; FILE *out; + int show_progress = isatty(2); /* variables to be filled by option parsing */ int pack_everything = 0; @@ -718,7 +719,7 @@ int cmd_repack(int argc, const char **argv, const char *prefix) } strbuf_release(&buf); } - if (!po_args.quiet && isatty(2)) + if (!po_args.quiet && show_progress) opts |= PRUNE_PACKED_VERBOSE; prune_packed_objects(opts);
-- 2.33.0.96.g73915697e6