D21409: KWrite: Implement skeleton KTextEditor::Application

[Christoph Cullmann]

cullmann added a comment.


  I think as an initial version this is ok.
  
  But you need to fix the list alteration during iteration
  
  for(auto kwrite: m_kwrites) {
  
        if (!kwrite->close()) {
            return false;
        }
    
        m_kwrites.removeAll(kwrite);
        delete kwrite;
    }

REPOSITORY
  R40 Kate

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

To: michalhumpula, cullmann
Cc: meven, anthonyfieroni, loh.tar, dhaumann, kwrite-devel, domson, michaelh, ngraham, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190605/cd4b27b9/attachment.html>