Review Request 122981: add new method KGlobalAccel::shortcut(componentName, actionId)

Thomas Lübking thomas.luebking at gmail.com
Thu Apr 16 20:56:31 UTC 2015



> On April 16, 2015, 3:20 nachm., Martin Gräßlin wrote:
> > src/kglobalaccel.h, line 276
> > <https://git.reviewboard.kde.org/r/122981/diff/5/?file=360654#file360654line276>
> >
> >     I think it needs a different name, see:
> >     https://techbase.kde.org/Policies/Binary_Compatibility_Issues_With_C++#The_Do.27s_and_Don.27ts
> >     
> >     in the don't section
> >     
> >     >  add an overload (BC, but not SC: makes &func ambiguous), adding overloads to already overloaded functions is ok (any use of &func already needed a cast).

What would be only relevant if we assume this to ever have become a function pointer target...

However, "globalShortcut" seems consistent enough ;-)


- Thomas


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


On April 10, 2015, 11:04 vorm., Gregor Mi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122981/
> -----------------------------------------------------------
> 
> (Updated April 10, 2015, 11:04 vorm.)
> 
> 
> Review request for KDE Frameworks, Martin Gräßlin and Thomas Lübking.
> 
> 
> Repository: kglobalaccel
> 
> 
> Description
> -------
> 
> Introduce new method: QList<QKeySequence> KGlobalAccel::shortcut(const QString& componentName, const QString& actionId) to retrieve shortcut list from global settings.
> 
> See discussion in https://git.reviewboard.kde.org/r/122249/ ("libksysguard: add Kill Window to End Process button and show correct keyboard shortcut").
> 
> 
> Diffs
> -----
> 
>   autotests/kglobalshortcuttest.cpp 3b661bbb612807a3bbbe34835d4ae712c2ec74da 
>   src/kglobalaccel.h 3fe20ca8e4ec6ceb0bb9e54235aef7f1aeeb8c16 
>   src/kglobalaccel.cpp 1b6b3f5cb6d42401d684e6a491d12a6e57248fd1 
>   src/kglobalaccel_p.h eca7c52378ad60d0d5806561214b9788dd46a11e 
>   autotests/kglobalshortcuttest.h b1122a8f5ca2f3f7afbe78f8edba87325426c1a6 
> 
> 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/20150416/2ad78db9/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list