ognarb updated this revision to Diff 52121.
ognarb added a comment.
Fix build
REPOSITORY
R8 Calligra
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D19132?vs=52119&id=52121
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D19132
AFFECTED FILES
CMakeLists.txt
lib
ognarb edited the summary of this revision.
REPOSITORY
R8 Calligra
REVISION DETAIL
https://phabricator.kde.org/D19132
To: ognarb, #calligra:_3.0, leinir
Cc: danders, Calligra-Devel-list, dcaliste, cochise, vandenoever
ognarb edited the summary of this revision.
ognarb edited the test plan for this revision.
REPOSITORY
R8 Calligra
REVISION DETAIL
https://phabricator.kde.org/D19132
To: ognarb, #calligra:_3.0, leinir
Cc: danders, Calligra-Devel-list, dcaliste, cochise, vandenoever
ognarb updated this revision to Diff 52119.
ognarb added a comment.
- Revert foreach modifications
- Fix some forgotten 0 as nullptr instances
REPOSITORY
R8 Calligra
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D19132?vs=52008&id=52119
BRANCH
master
REVISION DETAIL
https:
leinir added a comment.
There is an issue with Qt's foreach, however if the author would be so nice
as to check that this version using C++11 iteration does not break building on
Windows, then it can go in... Though, yes, it most certainly wants testing, and
i don't currently have a function
danders added a reviewer: leinir.
danders added a comment.
Hmmm. wasn't it somethng with ms windows and for loops, leinir?
Also, I think there may be detachment issues with them, clazy can tell.
There is a qAsConst() solution but that is not supported in the qt version we
need to support.