----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115251/#review48622 -----------------------------------------------------------
src/kstartupinfo.cpp <https://git.reviewboard.kde.org/r/115251/#comment34344> Does this need a runtime check? - David Edmundson On Jan. 27, 2014, 7:31 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115251/ > ----------------------------------------------------------- > > (Updated Jan. 27, 2014, 7:31 a.m.) > > > Review request for KDE Frameworks. > > > Repository: kwindowsystem > > > Description > ------- > > Add better runtime detection for X11 usage in KStartupInfo > > Compile time checks for X11 is no longer sufficient. This adds a > runtime check to all the code paths which look dangerous if executed > on a non-X11 platform. > > > Diffs > ----- > > src/kstartupinfo.cpp 5dbf47cb666fbed17c943491efe93e17f27d581e > > Diff: https://git.reviewboard.kde.org/r/115251/diff/ > > > Testing > ------- > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel