[Kde-finance-apps] Review Request 128361: Fix build with Qt 5.7

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Mon Jul 4 18:20:39 UTC 2016


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

Review request for Skrooge.


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


More information about the Kde-finance-apps mailing list