[PATCH] Core: use NGX_FILE_ERROR when appropriate

Valentin V. Bartenev vbart at nginx.com
Mon Mar 25 15:52:02 UTC 2013


On Monday 25 March 2013 18:41:28 Maxim Dounin wrote:
> Hello!
> 
> On Mon, Mar 25, 2013 at 06:15:56PM +0400, Valentin V. Bartenev wrote:
> > On Thursday 21 March 2013 08:04:24 Piotr Sikora wrote:
> > > Hey guys,
> > > attached patch changes "if" tests to use "== NGX_FILE_ERROR" instead of
> > > "== -1" or "!= NGX_OK", because NGX_FILE_ERROR is defined as -1 on
> > > UNIX, but as 0 on Win32.
> > 
> > The patch looks good. I'll commit it as soon as Maxim acknowledges.
> > Thank you.
> 
> Looks fine, please commit.

Done: http://trac.nginx.org/nginx/changeset/5138/nginx

  wbr, Valentin V. Bartenev



More information about the nginx-devel mailing list