On 26 Oct 2015, at 9:05 AM, Christophe JAILLET <christophe.jail...@wanadoo.fr> 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' and scanning 3 times for the > same thing is just a waste of time. > > Did I miss something obvious, or can it be turned in: > > #define CACHE_SEPARATOR ", “ Double check whether the spaces are significant, it might be a space and a tab (which would be weird, because it should just say /t then). This might have RFC compliance issues if it was changed, we need to verify what the code does. Regards, Graham —