<br><br><div><span class="gmail_quote">On 10/13/07, <b class="gmail_sendername">Oswald Buddenhagen</b> <<a href="mailto:ossi@kde.org">ossi@kde.org</a>> wrote:</span><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sat, Oct 13, 2007 at 09:52:25AM -0500, Thomas Braxton wrote:<br>> On 10/12/07, Oswald Buddenhagen <<a href="mailto:ossi@kde.org">ossi@kde.org</a>> wrote:<br>> > the escaping you chose is ... uhm ... ugly. i mean, even uglier than
<br>> > just tree backslashes in a row.<br>><br>> yeah, but 3 backslashes in a row doesn't work, \x5c does<br>><br>i think you *really* should re-read my first contribution to this thread<br>and everything that followed from it.
<br><br>> > anyway, a lot of the code looks suspicious, to say the least. i'll<br>> > know more when i actually get my hands on it.<br>><br>> like what?<br>><br>like all the special casing related to lists and separators. it's just
<br>plain weird. why don't you simply use a "forward parser" like<br>KConfigIniBackend::parseConfig() does instead of doing this incredible<br>magic?<br>i'm sure there is more. weirdness doesn't come alone. ;)
</blockquote><div><br class="webkit-block-placeholder"></div><div>Ok, give me a day or two. ;) even though this'll probably end up in KConfigIniBackend later anyway.</div><div><br class="webkit-block-placeholder"></div>
</div>