On Wed 25 Sep 2013 at 09:14:54PM +0200, Oswald Buddenhagen wrote: > i took a somewhat more aggressive and "creative" approach to the > problem, after all. please review and test the attached patches.
After testing the patches, I can confirm that my concerns have been
addressed: passwords that contain " and \ can be easily entered via
both Pass and PassCmd, and mailboxes with these characters are properly
parsed as well.
The only reservation I have is that all configuration arguments must now
backslash-escape literal " and \ regardless of whether or not they are
enclosed in double quotes. This is reminiscent of a shell's input parser
and is unlike string handling in most other kinds of configuration
files.
Since backwards compatibility will be broken either way, I think it
would be prudent to move to a simpler and more familiar string parsing
convention:
if arg begins with "
parse arg as double-quoted-string
else
read arg literally
This allows users to copy and paste ugly passwords like a\b"c\ into
their mbsyncrc without having to think about backslash escaping.
Another benefit is that this is the convention for generic configuration
syntax highlighting in many text editors, like the `conf` filetype in
vim, so the visual feedback will match the parser.
guns
pgpebFc0sKUXb.pgp
Description: PGP signature
------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60133471&iu=/4140/ostg.clktrk
_______________________________________________ isync-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/isync-devel
