Re: svn commit: r1683044 - /httpd/httpd/trunk/server/core.c

2015-06-04 Thread William A Rowe Jr
On Thu, Jun 4, 2015 at 1:23 PM, Marion Christophe JAILLET christophe.jail...@wanadoo.fr wrote: I agree that the wording of the Changelog could be more meaningful. Apparently these functions are only used during conf parsing. So, I propose to turn is into: Small speed optimization when

Re: svn commit: r1683044 - /httpd/httpd/trunk/server/core.c

2015-06-04 Thread Marion Christophe JAILLET
Hi, Skip a few bytes before calling 'strchr' if we know that they can't match. = in 'ap_resolve_env', at line 1265 we have: if (*s == '$') { if (s[1] == '{' (e = ap_strchr_c(s, '}'))) { So, we looking for an

Re: svn commit: r1683044 - /httpd/httpd/trunk/server/core.c

2015-06-03 Thread William A Rowe Jr
I tried to reconcile your patch with your svn log entry and I failed. Could you either correct or explain further? TIA, Bill On Jun 2, 2015 12:40 AM, jaillet...@apache.org wrote: Author: jailletc36 Date: Tue Jun 2 05:40:57 2015 New Revision: 1683044 URL: http://svn.apache.org/r1683044