Review Request 122981: add KGlobalAccel::loadShortcutFromGlobalSettings

Gregor Mi codestruct at posteo.org
Sat Mar 21 12:37:02 UTC 2015



> On March 17, 2015, 7:10 a.m., Martin Gräßlin wrote:
> > src/kglobalaccel.h, line 269
> > <https://git.reviewboard.kde.org/r/122981/diff/1/?file=355342#file355342line269>
> >
> >     what are the "some cases"?

I removed the "some cases". I was thinking about kwin/effects/desktopgrid/desktopgrid_config.cpp where the loadShortcutFromGlobalSettings() is not needed but there is setup might be different.


- Gregor


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122981/#review77606
-----------------------------------------------------------


On March 16, 2015, 8:29 p.m., Gregor Mi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122981/
> -----------------------------------------------------------
> 
> (Updated March 16, 2015, 8:29 p.m.)
> 
> 
> Review request for KDE Frameworks, Martin Gräßlin and Thomas Lübking.
> 
> 
> Repository: kglobalaccel
> 
> 
> Description
> -------
> 
> In some cases you need to call loadShortcutFromGlobalSettings() in order not to get a an empty list when calling shortcut(). See discussion in https://git.reviewboard.kde.org/r/122249/.
> 
> 
> Diffs
> -----
> 
>   src/kglobalaccel.h 3fe20ca8e4ec6ceb0bb9e54235aef7f1aeeb8c16 
>   src/kglobalaccel.cpp 1b6b3f5cb6d42401d684e6a491d12a6e57248fd1 
> 
> Diff: https://git.reviewboard.kde.org/r/122981/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gregor Mi
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150321/8065f9fb/attachment.html>


More information about the Kde-frameworks-devel mailing list