Re: [bug-gawk] Behavior of fflush with SIGPIPE on stdout [PATCH]

[Andrew J. Schorr]

Hi,

On Fri, Mar 17, 2017 at 08:44:35AM +0200, Arnold Robbins wrote:
> Thanks for the reports and discussions. I tackled the problem with the
> patch below. I plan to commit it soon.  Andy - please review.

I generally dislike that you are calling is_non_fatal_redirect() or
is_non_fatal_std() before calling fflush. You are incurring the overhead of the
PROCINFO lookup before the fflush call fails. Why take that performance hit
until we actually encounter an I/O error?

Does the change in SIGPIPE handling have anything to do with this issue,
or is that an unrelated code cleanup?

Regards,
Andy