D14952: Fix includedDefinitions, handle definition change in context switch
    Christoph Cullmann 
    noreply at phabricator.kde.org
       
    Mon Aug 20 16:52:46 BST 2018
    
    
  
cullmann created this revision.
cullmann added reviewers: vkrause, dhaumann.
cullmann added projects: Framework: Syntax Highlighting, Frameworks.
Herald added a project: Kate.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
cullmann requested review of this revision.
REVISION SUMMARY
  Fix includedDefinitions, handle definition change in context switch.
  They were missing, see PHP/HTML.
  I cleaned up the code a bit to just use QVector, no need for a queue, the order is anyways arbitrary (but stable).
TEST PLAN
  make && make test
  
  https://bugs.kde.org/show_bug.cgi?id=397659 works now
REPOSITORY
  R216 Syntax Highlighting
REVISION DETAIL
  https://phabricator.kde.org/D14952
AFFECTED FILES
  autotests/syntaxrepository_test.cpp
  src/lib/definition.cpp
To: cullmann, vkrause, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, genethomas, kevinapavew, ngraham, bruns, demsking, cullmann, vkrause, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180820/127fdf37/attachment.html>
    
    
More information about the KWrite-Devel
mailing list