[PATCH 2/2] pager: make wait_for_pager a no-op for "cat"
- From
Rubén Justo <rjusto@gmail.com>
- Date
- Jul 22, 2024, 23:24 UTC
- Message-ID
- <c37f0d54-4ead-422c-8193-f0c2ec84ca4a@gmail.com>
- In-Reply-To
- <43e045e5-4c92-4c5f-b183-d63c5b510023@gmail.com>
If we find that the configured pager is an empty string [*1*] or simply "cat" [*2*], then we return from `setup_pager()` silently without doing anything, allowing the output to go directly to the normal stdout.
Let's make the call to `wait_for_pager()` for these cases, or any other future optimizations that may occur, also exit silently without doing anything.
1.- 402461aab1 (pager: do not fork a pager if PAGER is set to empty.,
2006-04-16)2.- caef71a535 (Do not fork PAGER=cat, 2006-04-16)
Signed-off-by: Rubén Justo <rjusto@gmail.com> --- pager.c | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/pager.c b/pager.c index bea4345f6f..896f40fcd2 100644 --- a/pager.c +++ b/pager.c @@ -46,6 +46,9 @@ static void wait_for_pager_atexit(void) void wait_for_pager(void) { + if (old_fd1 == -1) + return; + finish_pager(); sigchain_pop_common(); unsetenv("GIT_PAGER_IN_USE");
-- 2.45.1