sirgienko added a comment.

  In D23061#509540 <https://phabricator.kde.org/D23061#509540>, @cullmann wrote:
  
  > Actually, wouldn't it make more sense to just provide a way to set the full 
keyword list?
  >  You can get the QStringlist will all keywords already with keywordList, a 
setKeywordList(...) would make more sense to me than this modifiers.
  >  That would allow to alter it like you want and then write it back in one 
go with just one re-init of the mapping.
  
  
  Yes, you right. I will change the pull request to `setKeywordList`.

REPOSITORY
  R216 Syntax Highlighting

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

To: sirgienko, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, univerz, LeGast00n, gennad, bmortimer, 
domson, michaelh, genethomas, ngraham, bruns, demsking, cullmann, vkrause, 
sars, dhaumann

Reply via email to