----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103469/#review9078 -----------------------------------------------------------
kdeui/kernel/kglobalsettings.cpp <http://git.reviewboard.kde.org/r/103469/#comment7509> I like the idea (KGlobalSettings reparsing configuration), but not the implementation (setLanguage(langage)). Can't KLocale get a reparseConfiguration() (to reuse the KConfig naming) ? - David Faure On Dec. 19, 2011, 2:02 p.m., Lamarque Vieira Souza wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103469/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2011, 2:02 p.m.) > > > Review request for kdelibs and Plasma. > > > Description > ------- > > This is patch number 3 to fix bug #289094 (top bar time incorrectly displays > in 24 hour format). The other patches are against plasma-mobile and > kde-workspace (http://git.reviewboard.kde.org/r/103434), all three patches > must be applied to fix the bug. I think this is a much simpler solution than > the one I suggested in http://git.reviewboard.kde.org/r/103434. > > > This addresses bug 289094. > http://bugs.kde.org/show_bug.cgi?id=289094 > > > Diffs > ----- > > kdeui/kernel/kglobalsettings.h cb8f7e2 > kdeui/kernel/kglobalsettings.cpp 819b314 > > Diff: http://git.reviewboard.kde.org/r/103469/diff/diff > > > Testing > ------- > > > Thanks, > > Lamarque Vieira Souza > >
