Author: Todd Lyons Date: To: Todd Lyons, Jeremy Harris, exim-dev Subject: Re: [exim-dev] Refactor optional MAIL args processing
On Thu, Apr 12, 2012 at 7:38 PM, Phil Pennock <pdp@???> wrote: >> > I did some testing and every other MTA I can find accepts a TAB as a valid
>> > separator, exim seems to be the only one who doesn't allow it. A quick fix
>> > doesn't appear to be too intrusive, but there may be better ways of solving
>> > it. This is in the extract_option() function. The line numbers will be
>> > off since I have some other modifications in this file:
> Went for a minor variation, using isspace(); the only bad character
> matched by that is \n which can't be present anyway. Seemed better than
> setting it to space, before an else check which wouldn't be applied
> anyway, only to set it to \0 a moment later. Two tests is more work
> than a 256 byte table lookup.
Understood. Passes my testing now. Thanks Phil!
Always code as if the guy who ends up maintaining your code will be a
violent psychopath who knows where you live. -- Martin Golding
This message was posted to the following mailing lists: