<table><tr><td style="">anthonyfieroni accepted this revision.<br />anthonyfieroni added a comment.<br />This revision is now accepted and ready to land.
</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/D16721">View Revision</a></tr></table><br /><div><div><p>Please, remove all white space changes before commit. It will be better if you test the patch some time without crash or other downsides. If you notice something wrong write here. If all is good you can commit in few days. Thank you for patch, other patches are welcome.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16721#inline-91388">View Inline</a><span style="color: #4b4d51; font-weight: bold;">KoToolBoxLayout_p.h:263</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">    <span class="p">}</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">
</div><div style="padding: 0 8px; margin: 0 4px; ">    <span class="n">QLayoutItem</span><span style="color: #aa2211">*</span> <span style="color: #004012">takeAt</span><span class="p">(</span><span style="color: #aa4000">int</span> <span class="n">i</span><span class="p">)</span> <span class="p">{</span> <span style="color: #aa4000">return</span> <span class="n">m_sections</span><span class="p">.</span><span class="n">takeAt</span><span class="p">(</span><span class="n">i</span><span class="p">);</span> <span class="p">}</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Remove all whitespace change, it doesn't do nothing.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16721">https://phabricator.kde.org/D16721</a></div></div><br /><div><strong>To: </strong>ognarb, Calligra: 3.0, anthonyfieroni<br /><strong>Cc: </strong>anthonyfieroni, Calligra-Devel-list, dcaliste, cochise, vandenoever<br /></div>