[Kde-graphics-devel] Review Request 122734: Fix build warnings - KRun::run deprecated for services, and unused variable.
Boudhayan Gupta
me at BaloneyGeek.com
Sat Feb 28 11:27:17 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122734/
-----------------------------------------------------------
(Updated Feb. 28, 2015, 11:27 a.m.)
Status
------
This change has been marked as submitted.
Review request for KSnapshot.
Repository: ksnapshot
Description
-------
In KF5, KRun::run is deprecated for services in favor of KRun::runService. This one-liner fixes that. Additionally, in the same line, the return value is saved to a bool variable which is checked with a Q_ASSERT macro; in Release builds this results in a warning about unused variables. This patch adds a Q_UNUSED macro after the Q_ASSERT macro to fix that.
Diffs
-----
ksnapshot.cpp fe9c1e2
Diff: https://git.reviewboard.kde.org/r/122734/diff/
Testing
-------
Build warnings are gone, no errors. Runs fine, takes screenshots fine. The 'Q' key shortcut to automatically take and save a screenshot also works.
My distribution (Arch) does not have KF5Kipi yet - all testing was done with Kipi disabled.
Thanks,
Boudhayan Gupta
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-graphics-devel/attachments/20150228/1e3b9219/attachment.html>
More information about the Kde-graphics-devel
mailing list