<br><br><div><span class="gmail_quote">On 10/8/07, <b class="gmail_sendername">Andreas Pakulat</b> <<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>> wrote:</span><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">
On 08.10.07 12:27:57, Thomas Braxton wrote:<br>> I know what was being pondered on this thread, but if I don't understand<br>> what you think is broken then I can't fix it can I?<br><br>Exactly what is tested in the test. KConfig doesn't properly store a
<br>list of strings if one of the strings has a \ at the end. I'm at the<br>moment not sure wether reading or writing or both are broken for this,<br>i.e. wether it doesn't escape the backslash or doesn't realize that this
<br>is not an escaped list-separator but in fact a list entry ending in a<br>backslash and a list separator afterwards.</blockquote><div><br class="webkit-block-placeholder"></div><div>does this work in KDE3? I can't test it right now, because I'm not at home, but if it doesn't why should we worry about it right now? It can be fixed later if/when KEntry is reworked to use QVariant instead of QByteArray. And if it's a Windows path then readPath/writePath should be used, now if that fails thats a *big* problem that needs to be fixed now.
</div><br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">Also as was proposed in this thread the list separator is not really<br>something the apps should decide to use or not as that makes no sense
<br>with backends that can store lists in a native format. (which is AFAIK<br>a big part of what all the kconfig rework was about)</blockquote><div><br class="webkit-block-placeholder"></div><div>I have no problem with this.
</div><div>Reworking KEntry to use QVariant instead of QByteArray will be alot of work, something I think can/should wait until after 4.0.</div><div> </div><br><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">
One note regarding the patch: Why are those flags removed from<br>kconfig_p.h? I'm just curious as I don't know the code very well anyway.</blockquote><div><br class="webkit-block-placeholder"></div><div>not really necessary, they were added to work around a bug somewhere else that I fixed in the patch.
</div><div><br class="webkit-block-placeholder"></div></div>