Review Request 128116: KStandardAction::showStatusbar: fix copy/paste error

Jonathan Marten jjm at keelhaul.me.uk
Tue Jun 7 10:53:44 UTC 2016


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

Review request for KDE Frameworks.


Repository: kconfigwidgets


Description
-------

This function is intended to return the action for the StandardAction of ShowStatusbar.  However, there seems to be an obvious copy/paste error here;  the action for ShowMenubar is retrieved instead.  This patch corrects this.


Diffs
-----

  src/kstandardaction.cpp 29b7e9b 

Diff: https://git.reviewboard.kde.org/r/128116/diff/


Testing
-------

Built tier3/kconfigwidgets with this change, and tested with an application which looks up the action by name.  Without this change there is a segfault because the wrong action is found;  with the change the correct action is found and works.

The autotest does not detect this problem because it only checks that an action is returned; it doesn't check that it is the correct one!  Fixing this would need an extra test of each returned action to check that its name() is correct - not sure whether this is necessary?


Thanks,

Jonathan Marten

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160607/4205435f/attachment.html>


More information about the Kde-frameworks-devel mailing list