<br><br><div><span class="gmail_quote">On 10/17/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 Wed, Oct 17, 2007 at 04:54:09AM -0500, Thomas Braxton wrote:<br>> Personally I think this is uglier than hex encoding the trailing<br>> backslash.<br>><br>- encoding just a trailing backslash is sicko. and it is how the windows

<br>  shell works wrt quotes, so it can't be good. ;)</blockquote><div><br class="webkit-block-placeholder"></div><div>i wouldn't know, i stay away from windows as much as possible. </div><br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">

- hex-coding is "a bit" inconsistent with the rest of the escaping.<br>  talking surprises ...</blockquote><div><br class="webkit-block-placeholder"></div><div>not my fault windows uses our escape char as a dir separator, but it's the only way i could figure out how to let KConfigGroup know that backslash is special

</div><br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">> Since we can't know if this is a trailing backslash or escaping the<br>> separator, every backslash has to be escaped twice. Once by

<br>> KConfigGroup, and once by KConfigIniBackend<br>><br>correct. i said this quite some mails before already.<br><br>> And I'm not sure, but I think this will cause problems if/when we<br>> change KEntry to use QVariant instead of QByteArray, because there's

> no way for KConfigIniBackend to know if it should unescape the > backslashes once or twice. > you would have this problem either way as long as the backend does not know whether it is dealing with a list in the first place.

</blockquote><div><br class="webkit-block-placeholder"></div><div>KConfigIniBackend will never *know* it is dealing with a list, it can only guess, unless we start saving the type in the file.</div><br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">

this just shows that the refactoring suggested near the start of this<br>thread is a must-have. we won't get anywhere without it, let alone have<br>the possibility to add other backends.</blockquote><div><br class="webkit-block-placeholder">

</div><div>I'm sure everyone is ready to wait while we iron out the bugs from changing KEntry to using QVariant instead of QByteArray. Without another backend this is alot of something for nothing</div><br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">

--<br>Hi! I'm a .signature virus! Copy me into your ~/.signature, please!<br>--<br>Chaos, panic, and disorder - my work here is done.<br></blockquote></div><br>