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

Jonathan Marten jjm at keelhaul.me.uk
Tue Jun 7 12:45:22 UTC 2016


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

(Updated June 7, 2016, 2:45 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Changes
-------

Submitted with commit 3f7465b8d7c1649b54eb6ce72e88d2a3080e2234 by Jonathan Marten to branch master.


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/548290a0/attachment.html>


More information about the Kde-frameworks-devel mailing list