On Thu, Aug 22, 2019 at 11:40:15AM +0200, Patrick Steinhardt wrote:
> Note that both `close_stdout` and `close_stdout_atexit` are misnamed
> after this change, as we do not actually close the streams now. In order
> to avoid unnecessary code churn, we still retain their current names.
Yes, it would be nice to rename it later to something like
"flush_stdstreams" or "verify_stdstreams" or so...
> include/closestream.h | 33 ++++++++++++++++++++++++++++-----
> 1 file changed, 28 insertions(+), 5 deletions(-)
Looks good. I'll apply the patch later (let's wait for another reviews).
And it seems we can remove
#if !defined(__SANITIZE_ADDRESS__)
from the code now.
Karel
--
Karel Zak <kzak@xxxxxxxxxx>
http://karelzak.blogspot.com
