On Fri, Jan 19, 2018 at 9:33 AM, <randall.s.becker@xxxxxxxxxx> wrote:
> From: "Randall S. Becker" <rsbecker@xxxxxxxxxxxxx>
>
> * wrapper.c: called setbuf(stream,0) to force pipe flushes not enabled by
> default on the NonStop platform.
>
Due to my review of a previous patch, I now know about the __TANDEM
directive and why we use this to guard this line. :)
The setbuf(0) is easier than sprinkling (guarded) flushes all over the code,
so that seems like a clean solution, which we currently don't use.
(it occurred twice in history, see eac14f8909 (Win32: Thread-safe
windows console output, 2012-01-14) for its introduction and fcd428f4a9
(Win32: fix broken pipe detection, 2012-03-01) for its deletion).
> Signed-off-by: Randall S. Becker <rsbecker@xxxxxxxxxxxxx>
> ---
> wrapper.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/wrapper.c b/wrapper.c
> index d20356a77..671cbb4b4 100644
> --- a/wrapper.c
> +++ b/wrapper.c
> @@ -403,6 +403,9 @@ FILE *xfdopen(int fd, const char *mode)
> FILE *stream = fdopen(fd, mode);
> if (stream == NULL)
> die_errno("Out of memory? fdopen failed");
> +#ifdef __TANDEM
> + setbuf(stream,0);
My man page tells me the second arg is a pointer,
so we'd prefer NULL instead?
Thanks,
Stefan
> +#endif
> return stream;
> }
>
> --
> 2.16.0.31.gf1a482c
>
