leinir accepted this revision.
leinir added a comment.
This revision is now accepted and ready to land.
Oh dear, yes, took a moment. Well spotted indeed!
REPOSITORY
R8 Calligra
REVISION DETAIL
https://phabricator.kde.org/D23492
To: dcaliste, leinir, pvuorela, danders, boemann
Cc: Calligr
pvuorela added a comment.
Indeed that doesn't look like a good loop, if triggered once, nothing
affecting the break out condition inside of it. But isn't the same thing
happening also at the beginning of this method, and actually same kind of
construct being there in spreadsheet and presenta