[Kde-finance-apps] Review Request 128361: Fix build with Qt 5.7
Martin Tobias Holmedahl Sandsmark
martin.sandsmark at kde.org
Mon Jul 4 19:03:34 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128361/
-----------------------------------------------------------
(Updated July 4, 2016, 7:03 p.m.)
Status
------
This change has been marked as submitted.
Review request for Skrooge.
Changes
-------
Submitted with commit 1ac4c788019cb3eb6e8fbd2caccae881fe115e48 by Martin T. H. Sandsmark to branch master.
Repository: skrooge
Description
-------
I assume that in earlier versions of Qt qAsConst() didn't exist, that's why it was declared locally, so now don't declare it on Qt 5.7.
In addition in Qt 5.7 it is not possible to use qAsConst() with variables that are already const, so remove one usage of qAsConst where a variable was already const.
Diffs
-----
skgbasemodeler/skgdefine.h 5c0a207
skgbasemodeler/skgdocument.cpp c4b8947
Diff: https://git.reviewboard.kde.org/r/128361/diff/
Testing
-------
It now builds with Qt 5.7.
Thanks,
Martin Tobias Holmedahl Sandsmark
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-finance-apps/attachments/20160704/461e48f2/attachment.html>
More information about the Kde-finance-apps
mailing list