https://issues.apache.org/SpamAssassin/show_bug.cgi?id=5895
--- Comment #25 from Mark Martinec <[email protected]> 2010-01-06 03:20:22 UTC --- > Oh, I didn't notice that change in the split from /\s+/ to ' ' > > I agree that it should be /\s+/ to be consistent with every other such config > option. I'll apply the patch but keep the /\s+/. As for consistency, some code splits on ' ' (like set_addrlist_value), some does on /\s+/, there is no consistency right now, it would be nice to fix it some day. > It would not be good for someone to use tabs in their config file and > then have it break on that one option when it has always worked for them. This is a misinterpretation on what split(' ') does. There is no difference as far as tabs are concerned: man perlfunc: As a special case, specifying a PATTERN of space (' ') will split on white space just as "split" with no arguments does. [...] A "split" on "/\s+/" is like a "split(' ')" except that any leading whitespace produces a null first field. -- Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.
