Subject: Re: [PATCH 2/5] pager: do not close fd 2 unnecessarily

From: Junio C Hamano <[email-masked]>
Date: Tue, 21 May 2024 15:00:10 -0700

Rubén Justo <[email-masked]> writes:

> Adding logic to adjust when we close(stderr) in close_pager_fds() makes
> sense on its own, I think.

The feature may be.

> And, the values for the flag "do-we-want-to-close-stderr-at-exit", too,
> to me.

But the thing is, the flag is *NOT* named as such, and an
undocumented "value -1 means X, while value 1 means Y", do not make
any sense, either.