D23034: implicit fallthough if there is fallthoughContext

[Dominik Haumann]

dhaumann requested changes to this revision.
dhaumann added a comment.


  I like this change a lot, but there is a huge (theoretical) backwards compatibility issue with this.
  
  Before, the code ignored `fallthroughContext` if `m_fallthrough` was explicitly set to `false`. So you could have: fallghrough="false" fallthroughContext="abc", and fallthroughContext was ignored.
  Now this changed, and `m_fallthough` solely depends on whether `fallthroughContext` is a valid context or not other than `#stay`.
  
  My point is: For changes like this, we usually have to
  
  - increase the `version` number in the `*.xml` files
  - increase the `kateversion` from `5.0` to `5.62`?
  - Since we have `IncludeRules`, wouldn't we even need to increase the `kateversion` number of _all_ xml files to be on the safe side?
  
  I think these issues first need to be clarified before we can commit this.

REPOSITORY
  R216 Syntax Highlighting

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

To: jpoelen, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, univerz, LeGast00n, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190809/f5a13135/attachment-0001.html>