On Sun, 11 Sep 2016 21:59:12 +0200
Giuseppe Scrivano <[email protected]> wrote:

> Matthew White <[email protected]> writes:
> 
> > +              */
> > +              if (!output_stream && (output_stream = fopen (filename, 
> > "rb")))
> > +                {
> > +                  fclose (output_stream);
> > +                  output_stream = fopen (filename, "ab");
> > +                }
> 
> please use file_exists_p instead of opening and closing the file:
> 
> if (!output_stream && file_exists_p (output_stream))
>    output_stream = fopen (filename, "ab");

Thanks Giuseppe.

Amended with 'if (!output_stream && file_exists_p (filename))'.

Posting after final decisions are taken about open topics in this series of 
patches.

> 
> Giuseppe

Regards,
Matthew

-- 
Matthew White <[email protected]>

Attachment: pgpQEaQaRyApj.pgp
Description: PGP signature

Reply via email to