Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-17 Thread André Malo
* Rainer Jung wrote: On 27.03.2013 22:57, jaillet...@apache.org wrote: Author: jailletc36 Date: Wed Mar 27 21:57:44 2013 New Revision: 1461869 URL: http://svn.apache.org/r1461869 Log: Be more clever when allocating memory for log item to be escaped. This should save about 70-100

Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-17 Thread André Malo
* André Malo wrote: * Rainer Jung wrote: A quick draft patch is at http://people.apache.org/~rjung/patches/ap_escape_logitem_enhanced.patc h +ret = apr_palloc(p, length + 3 * escapes); It should be still 4 * escapes ('\xHH'). Also, escapes should be apr_size_t as well, I think.

RE: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-17 Thread Plüm , Rüdiger , Vodafone Group
-Original Message- From: André Malo [mailto:n...@perlig.de] Sent: Mittwoch, 17. April 2013 11:04 To: dev@httpd.apache.org Subject: Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c * Rainer Jung wrote: http://people.apache.org/~rjung/patches

Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-17 Thread Rainer Jung
On 17.04.2013 11:07, André Malo wrote: * André Malo wrote: * Rainer Jung wrote: A quick draft patch is at http://people.apache.org/~rjung/patches/ap_escape_logitem_enhanced.patc h +ret = apr_palloc(p, length + 3 * escapes); It should be still 4 * escapes ('\xHH'). Hmmm, isn't the

Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-17 Thread Rainer Jung
On 17.04.2013 11:38, Plüm, Rüdiger, Vodafone Group wrote: -Original Message- From: André Malo [mailto:n...@perlig.de] Sent: Mittwoch, 17. April 2013 11:04 To: dev@httpd.apache.org Subject: Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c * Rainer Jung wrote: http

Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-17 Thread André Malo
* Rainer Jung wrote: On 17.04.2013 11:38, Plüm, Rüdiger, Vodafone Group wrote: -Original Message- From: André Malo [mailto:n...@perlig.de] Sent: Mittwoch, 17. April 2013 11:04 To: dev@httpd.apache.org Subject: Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

Re: svn commit: r1461869 - /httpd/httpd/trunk/server/util.c

2013-04-16 Thread Rainer Jung
On 27.03.2013 22:57, jaillet...@apache.org wrote: Author: jailletc36 Date: Wed Mar 27 21:57:44 2013 New Revision: 1461869 URL: http://svn.apache.org/r1461869 Log: Be more clever when allocating memory for log item to be escaped. This should save about 70-100 bytes in the request pool with