[PATCH] Continuation lines
Holger Weiss
holger at CIS.FU-Berlin.DE
Sun Dec 9 20:30:19 CET 2007
* Ethan Galstad <nagios at nagios.org> [2007-12-09 10:53]:
> Holger Weiss wrote:
> > The attached patch adds support for using continuation lines throughout
> > all configuration files: If the last character before a newline is a
> > backslash, the backslash will be stripped, and if the second last
> > character is not a backslash, the following line will be appended prior
> > to parsing.
> >
> > base/utils.c (and cgi/cgiutils.c) had an mmap_fgets_multiline() function
> > already, but the function wasn't used anywhere and it didn't work out of
> > the box.
>
> Thanks for the note and patch Holger. I must have forgotten to finish
> implementing this in the past. I fixed the bug in the original function
> and have posted the fixes to CVS.
Thanks a lot, though the following line is now appended if and only if
two backslashes are used. From the original function, I assumed the
idea was to append the following line if the current line ends with a
backslash _unless_ this backslash was escaped using another backslash.
IMO, doing it this way round would be more intuitive as it's a common
way to implement continuation lines.
Holger
-------------------------------------------------------------------------
SF.Net email is sponsored by:
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
More information about the Developers
mailing list