Re: svn commit: r1588704 - in /httpd/httpd/trunk: CHANGES modules/cache/cache_util.c

2014-04-20 Thread Marion Christophe JAILLET
Hi, The description and the CHANGES are about AH00784 but the patch is only about adding 'status' to AH00783. -else if (APR_EEXIST == status) { +else if (APR_STATUS_IS_EEXIST(status)) { seems not to change anything. #define APR_STATUS_IS_EEXIST(s) ((s) == APR_EEXIST) So, is

Re: svn commit: r1588704 - in /httpd/httpd/trunk: CHANGES modules/cache/cache_util.c

2014-04-20 Thread Christophe JAILLET
Le 20/04/2014 08:52, Marion Christophe JAILLET a écrit : Hi, The description and the CHANGES are about AH00784 but the patch is only about adding 'status' to AH00783. -else if (APR_EEXIST == status) { +else if (APR_STATUS_IS_EEXIST(status)) { seems not to change anything. #define

Re: svn commit: r1588704 - in /httpd/httpd/trunk: CHANGES modules/cache/cache_util.c

2014-04-20 Thread Eric Covener
On Sun, Apr 20, 2014 at 3:07 AM, Christophe JAILLET christophe.jail...@wanadoo.fr wrote: Sorry for the noise. Review appreciated nonetheless.