D19532: Don't crash when no contexts are present

Volker Krause noreply at phabricator.kde.org
Wed Mar 6 17:17:23 GMT 2019


vkrause added a comment.


  In D19532#425432 <https://phabricator.kde.org/D19532#425432>, @cullmann wrote:
  
  > Ok with that.
  >  Volker, ok, too?
  >  I think the if(...isLoaded) removal is a diff artifact, that was never in the original code bug just in the first try here.
  
  
  Yep, replacing the assert certainly makes sense. This would assume though that the caller of initalContext() expects a nullptr return value already, which I guess is the case if this doesn't crash somewhere else now :)

REPOSITORY
  R216 Syntax Highlighting

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

To: davispuh, cullmann, dhaumann, vandenoever, vkrause
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/20190306/3f7be58a/attachment.html>


More information about the KWrite-Devel mailing list