|This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103058/|
On November 7th, 2011, 5:35 p.m., Martin Gräßlin wrote:
kwin/options.cpp (Diff revision 1) int currentRefreshRate() 67 if (!ok) 68 frate = QLocale::system().toFloat(reply, &ok); 69 if (!ok) 70 frate = QLocale(QLocale::German).toFloat(reply, &ok); // "," 71 if (!ok) 72 frate = QLocale(QLocale::English).toFloat(reply, &ok); // "."meh that doesn't look nice. Is there no better way to get it? Let me guess: the language from nvidia_settings does not have to be the same as returned by QLocale::system()?
On November 9th, 2011, 10:06 p.m., Thomas Lübking wrote:LOCAL(E) issue - for some reason, kwin thinks the locale is "C" but nvidia-settings thinks it's de_DE.UTF-8 (as set in the boot var) - might be because arch recently changed the system and KDM doesn't read in wherever this ends (i just blindly followed their instructions ;-) As a broadsword solution, the coming patches set the locale for the nvidia-settings process environment (to "C") and then QLocale::c().toFloat() is used. This works always ;-) (Note to self: as soon as world domination reached, enforce common global language. <- there's no smiley)
On November 12th, 2011, 6:48 p.m., Fredrik Höglund wrote:Does NVIDIA support GLX_OML_sync_control? If it does you can get the refresh rate with glXGetMscRateOML().
On November 12th, 2011, 7:15 p.m., Thomas Lübking wrote:nope - not on this board (but it is the main driver, not the legacy one)
Just checked on my NVIDIA system (running 275.28) for the extension and result is negative
On November 10th, 2011, 4:30 p.m., Thomas Lübking wrote:
Review request for kwin and Martin Gräßlin.
By Thomas Lübking.
Updated Nov. 10, 2011, 4:30 p.m.
|Free embeddable forum powered by Nabble||Forum Help|