<table><tr><td style="">loh.tar created this revision.<br />loh.tar added a reviewer: KTextEditor.<br />Herald added projects: Kate, Frameworks.<br />Herald added subscribers: kde-frameworks-devel, kwrite-devel.<br />loh.tar requested review of this revision.
</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/D19446">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>..how it should be by adding brackets to each line of the block</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Encapsulate in editBegin/End</li>
<li class="remarkup-list-item">Reduce if/else nesting</li>
</ul>

<p>BUG:382213<br />
Fixed in: 5.57</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Oops?! I'm sorry the diff looks terrible. In the past were leading space changes ignored, so I had done these "Reduce if/else nesting" :-/</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Feature or bug? Select in block mode from right->left on a single line after last char a block. The brackets are exchanged )( and placed at the end of the block</li>
<li class="remarkup-list-item">Code use MovingRange, old(?) similar code fumble around with backup cursors, I hope not for some optimizing but only because its older than MovingRange</li>
<li class="remarkup-list-item">No use of toVirtualColumn or similar. It looks so far good anyway. Only when you mix tabs and spaces the result may sometimes odd. But I think there is no solution in using toVirtualColumn in the cases I had played around</li>
<li class="remarkup-list-item">No call of view->slotTextInserted (will only emit signal) can't see any effect and isn't such signal not already emit by the doc itself?</li>
<li class="remarkup-list-item">Autotests looks OK, I may add some for the new feature when patch gets accepted so far</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19446">https://phabricator.kde.org/D19446</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/document/katedocument.cpp</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, KTextEditor, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>