----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118985/#review61112 -----------------------------------------------------------
src/core/ksharedconfig.cpp <https://git.reviewboard.kde.org/r/118985/#comment42559> -- slightly unrelated since it wasn't there before this patch. } else { wasTestModeEnabled = false; } ? I don't know this code, but what happens here is wasTestModeEnabled is initialized to false, then set to true if it's false and isTestModeEnabled is true. Fine, but it's never set back to false again. I don't know if it's an issue at all.. Just something i noticed. The same for the "userWarned" a few lines down. - Mark Gaiser On June 27, 2014, 10:31 p.m., David Faure wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118985/ > ----------------------------------------------------------- > > (Updated June 27, 2014, 10:31 p.m.) > > > Review request for KDE Frameworks and Matthew Dawson. > > > Repository: kconfig > > > Description > ------- > > KSharedConfig: only write to mainConfig and wasTestModeEnabled in the main > thread > > As a side effect, this removes the need for the atomic ints, since > that code now only runs in the main thread. > > > Diffs > ----- > > src/core/ksharedconfig.cpp b7d155d5893502921d35d7dd971188b6a93a0620 > > Diff: https://git.reviewboard.kde.org/r/118985/diff/ > > > Testing > ------- > > no regression in "make test" in kconfig; still debugging races in helgrind > threadtest (in kio). > > > Thanks, > > David Faure > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
