<br><br><div><span class="gmail_quote">On 10/9/07, <b class="gmail_sendername">Nhuh Put</b> <<a href="mailto:nhuh.put@web.de">nhuh.put@web.de</a>> wrote:</span><blockquote class="gmail_quote" style="margin:0;margin-left:0.8ex;border-left:1px #ccc solid;padding-left:1ex">
Von: Thomas Braxton<br>Gesendet: Dienstag, 9. Oktober 2007 21:22<br>An: <a href="mailto:kde-core-devel@kde.org">kde-core-devel@kde.org</a><br>Betreff: Re: KConfig/KLockFile performance (was KMimeType performance)<br><br>> On 10/9/07, David Faure <
<a href="mailto:faure@kde.org">faure@kde.org</a>> wrote:<br>> On Tuesday 09 October 2007, Maksim Orlovich wrote:<br>> > KDesktopFile/KConfig tries to lock the file with KLockFile<br>><br>> Huh. I thought we got rid of the "bool readonly" because it made no behavior difference,
<br>> but if KConfig locks the file then surely we need a "ReadOnly" flag again?<br>> Or better: we should lock at the time of writing to the file, not when just reading from it!<br>><br>> your mail came in as I was sending the last patch, here's another try. this time only locking in sync() and only if the file is read-write.
<br><br>This should work, but there is one problem: configState is only set to ReadOnly in<br>KConfig::isConfigWritable, which is normally not called, so there should probably be<br>a call to it in KConfigPrivate::changeFileName
, or this will not have much effect.</blockquote><div><br class="webkit-block-placeholder"></div><div>I thought I already made that change, I must have forgot to commit it, probably when I was having trouble getting a good update and had to get a clean checkout 
</div><br></div><br>