On Thu, Apr 12, 2012 at 7:38 PM, Phil Pennock <[email protected]> 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! ...Todd -- Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live. -- Martin Golding -- ## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##
