D24908: fix default shortcut detection

Albert Astals Cid noreply at phabricator.kde.org
Thu Oct 24 23:42:38 BST 2019


aacid accepted this revision.
aacid added a comment.
This revision is now accepted and ready to land.


  ok, fair enough about the tests

INLINE COMMENTS

> sitter wrote in kshortcuteditwidget.cpp:72
> it's initialized to None via the ctor at line 68 already. now that I notice it, defaultText could also be const now.

ah right.

REPOSITORY
  R263 KXmlGui

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D24908

To: sitter, dfaure, aacid
Cc: aacid, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191024/63034e35/attachment.html>


More information about the Kde-frameworks-devel mailing list