cullmann added a comment.
I like the idea. For the patch: I think one should check with isEmpty() if the thing is empty, instead of the size() > 0 things, thats more clear. And is there no m_currentAutobraceClosingChar clear missing somewhere? e.g. don't go this two stack out of sync? REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D12295 To: sraizada, #ktexteditor, #kate, cullmann, dhaumann Cc: ngraham, kwrite-devel, kde-frameworks-devel, #ktexteditor, michaelh, kevinapavew, bruns, demsking, head7, cullmann, kfunk, sars, dhaumann
