D14236: Add some improvements to kate-syntax-highlighter for use in scripting

Christoph Cullmann noreply at phabricator.kde.org
Thu Aug 30 08:53:04 BST 2018


cullmann added a comment.


  Thanks for the enhancements!
  
  I just did hide the bestCandidate function as static helper, no need to expose them at all in the header:
  
  Git commit 60affa47908782d2bda94af2e49fa3f1c4608f4e <https://phabricator.kde.org/R216:60affa47908782d2bda94af2e49fa3f1c4608f4e> by Christoph Cullmann.
  Committed on 30/08/2018 at 07:51.
  Pushed by cullmann into branch 'master'.
  
  hide bestCandidate, can be static function inside file
  
  M  +12   -12   src/lib/repository.cpp
  M  +0    -2    src/lib/repository.h
  
  https://commits.kde.org/syntax-highlighting/60affa47908782d2bda94af2e49fa3f1c4608f4e

REPOSITORY
  R216 Syntax Highlighting

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

To: xciml, #framework_syntax_highlighting, vkrause, cullmann
Cc: cullmann, pino, kde-frameworks-devel, kwrite-devel, #framework_syntax_highlighting, bmortimer, michaelh, genethomas, kevinapavew, ngraham, bruns, demsking, vkrause, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180830/e4ec0ab4/attachment.html>


More information about the Kde-frameworks-devel mailing list