dhaumann added a comment.

  I think this change introduces a regression: This code does not check for 
"view->config()->autoBrackets()", so it's *ALWAYS* active, which I do not want. 
Consider this:
  
    if (bla |) { ... }
  
  Now you type && foo(), the result is then:
  
    if (bla && foo() { ... }
  
  Notice the unbalanced closing ), which is due to this patch.
  
  Can we either revert this, or at least add config check?

REPOSITORY
  R39 KTextEditor

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

To: loh.tar, #ktexteditor, cullmann, mwolff
Cc: dhaumann, meven, kwrite-devel, kde-frameworks-devel, #ktexteditor, domson, 
michaelh, ngraham, bruns, demsking, cullmann, sars

Reply via email to