Question about CACHE_SEPARATOR in modules/cache/cache_util.h

2015-10-26 Thread Christophe JAILLET
Hi, in modules/cache/cache_util.h, CACHE_SEPARATOR is defined as: #define CACHE_SEPARATOR ", " I don't see any reason to have 3 spaces here. It is only used within calls to 'cache_strqtok' and scanning 3 times for the same thing is just a waste of time. Did I miss something obvious,

RE: Question about CACHE_SEPARATOR in modules/cache/cache_util.h

2015-10-26 Thread Plüm , Rüdiger , Vodafone Group
> -Original Message- > From: Christophe JAILLET [mailto:christophe.jail...@wanadoo.fr] > Sent: Montag, 26. Oktober 2015 08:06 > To: dev@httpd.apache.org > Subject: Question about CACHE_SEPARATOR in modules/cache/cache_util.h > > Hi, > > in modules/cache/ca

Re: Question about CACHE_SEPARATOR in modules/cache/cache_util.h

2015-10-26 Thread Graham Leggett
On 26 Oct 2015, at 9:05 AM, Christophe JAILLET wrote: > in modules/cache/cache_util.h, CACHE_SEPARATOR is defined as: > > #define CACHE_SEPARATOR ", " > > > I don't see any reason to have 3 spaces here. > It is only used within calls to 'cache_strqtok'

Re: Question about CACHE_SEPARATOR in modules/cache/cache_util.h

2015-10-26 Thread Graham Leggett
On 26 Oct 2015, at 2:15 PM, Rainer Jung wrote: > The line goes back to > > https://bz.apache.org/bugzilla/show_bug.cgi?id=50199 > > where Nick reported the problem and Graham provided the initial patch. > > The defined tokenizer chars were "comma-space-space-space"

Re: Question about CACHE_SEPARATOR in modules/cache/cache_util.h

2015-10-26 Thread Rainer Jung
Am 26.10.2015 um 10:11 schrieb Graham Leggett: On 26 Oct 2015, at 9:05 AM, Christophe JAILLET wrote: in modules/cache/cache_util.h, CACHE_SEPARATOR is defined as: #define CACHE_SEPARATOR ", " I don't see any reason to have 3 spaces here. It is only