----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124902/#review84264 -----------------------------------------------------------
src/kglobalaccel.cpp (line 97) <https://git.reviewboard.kde.org/r/124902/#comment58351> No non trivial global static in a lib? - Kai Uwe Broulik On Aug. 24, 2015, 11:35 vorm., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124902/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2015, 11:35 vorm.) > > > Review request for KDE Frameworks and Kevin Funk. > > > Repository: kglobalaccel > > > Description > ------- > > There are usages of KGlobalAccel in e.g. KActionCollection of xmlgui > framework which do not require a running kglobalaccel5. But due to the > way KGlobalAccel inits, it always started the runtime part. > > This change ensures that kglobalaccel5 is only started once it's actually > needed, by e.g. registering a global shortcut. If an application does > not use global shortcuts the API should no longer trigger a start of the > runtime part. > > > Diffs > ----- > > src/kglobalaccel_p.h a84f4038dac650ddb0fc1828e67e0851153b4456 > src/kglobalaccel.cpp 3508c870c70a21e76263b7574c20b408cc09a837 > > Diff: https://git.reviewboard.kde.org/r/124902/diff/ > > > Testing > ------- > > export $(dbus-launch) > qdbusviewer & > kate & > > -> kglobalaccel5 gets started > > with the change: > kglobalaccel5 no longer gets started, when starting an application using > global shortcuts, it still gets started > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
