D19598: DocumentPrivate: Make auto bracket slightly smarter

loh tar noreply at phabricator.kde.org
Thu Mar 7 17:07:19 GMT 2019


loh.tar created this revision.
loh.tar added a reviewer: KTextEditor.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
loh.tar requested review of this revision.

REVISION SUMMARY
  ..in handling of quotes
  
  - Skip auto quotes when these looks already balanced
  - Improve readability, const correctness
  - const QChar as value, not reference
  
  BUG:405089
  FIXED-IN: 5.56

TEST PLAN
    Try to type this in a C++ file:
    ('\'') => ended up in something like: ('\''')
    
    Try to add afterwarnds double quotes to some word in a text, without to select the word:
    foo "bar" haz => ended up in something like: foo "bar"" haz
  
  - Autotest looks good
  - Contains partial code from unloved D17693 <https://phabricator.kde.org/D17693> which is here only added to became it in. See latter two Summary notes
  - Is the fix-in number right?

REPOSITORY
  R39 KTextEditor

REVISION DETAIL
  https://phabricator.kde.org/D19598

AFFECTED FILES
  src/document/katedocument.cpp

To: loh.tar, #ktexteditor
Cc: kwrite-devel, kde-frameworks-devel, #ktexteditor, gennad, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190307/2dd32f2b/attachment.html>


More information about the KWrite-Devel mailing list