<table><tr><td style="">pvuorela added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D23492">View Revision</a></tr></table><br /><div><div><p>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 presentation impls too?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23492">https://phabricator.kde.org/D23492</a></div></div><br /><div><strong>To: </strong>dcaliste, leinir, pvuorela, danders, boemann<br /><strong>Cc: </strong>Calligra-Devel-list, Calligra: 3.0, dcaliste, cochise, vandenoever<br /></div>