D19532: Don't crash when no contexts are present

Christoph Cullmann noreply at phabricator.kde.org
Tue Mar 5 19:21:21 GMT 2019


cullmann added a comment.


  In D19532#425384 <https://phabricator.kde.org/D19532#425384>, @davispuh wrote:
  
  > In D19532#425381 <https://phabricator.kde.org/D19532#425381>, @cullmann wrote:
  >
  > > Calling the ::get is no issue.
  >
  >
  > That's good then, but we can't use `applyFormat(0, text.size(), Format());` because Kate asserts when `!Format->isValid()` which it won't be so we have to  `return State();` and also see D19533 <https://phabricator.kde.org/D19533>
  
  
  :=) I guess we just need to if (!format.isValid()) return; in the KTextEditor code, seems I didn't read the docs of applyFormat, it just works by chance ATM as I filter out !isValid() definitions there ATM.

REPOSITORY
  R216 Syntax Highlighting

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

To: davispuh, cullmann, dhaumann, vandenoever
Cc: kwrite-devel, kde-frameworks-devel, 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/20190305/f0f127cd/attachment-0001.html>


More information about the KWrite-Devel mailing list