<table><tr><td style="">dhaumann requested changes to this revision.<br />dhaumann added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D23034">View Revision</a></tr></table><br /><div><div><p>I like this change a lot, but there is a huge (theoretical) backwards compatibility issue with this.</p>

<p>Before, the code ignored <tt style="background: #ebebeb; font-size: 13px;">fallthroughContext</tt> if <tt style="background: #ebebeb; font-size: 13px;">m_fallthrough</tt> was explicitly set to <tt style="background: #ebebeb; font-size: 13px;">false</tt>. So you could have: fallghrough="false" fallthroughContext="abc", and fallthroughContext was ignored.<br />
Now this changed, and <tt style="background: #ebebeb; font-size: 13px;">m_fallthough</tt> solely depends on whether <tt style="background: #ebebeb; font-size: 13px;">fallthroughContext</tt> is a valid context or not other than <tt style="background: #ebebeb; font-size: 13px;">#stay</tt>.</p>

<p>My point is: For changes like this, we usually have to</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">increase the <tt style="background: #ebebeb; font-size: 13px;">version</tt> number in the <tt style="background: #ebebeb; font-size: 13px;">*.xml</tt> files</li>
<li class="remarkup-list-item">increase the <tt style="background: #ebebeb; font-size: 13px;">kateversion</tt> from <tt style="background: #ebebeb; font-size: 13px;">5.0</tt> to <tt style="background: #ebebeb; font-size: 13px;">5.62</tt>?</li>
<li class="remarkup-list-item">Since we have <tt style="background: #ebebeb; font-size: 13px;">IncludeRules</tt>, wouldn't we even need to increase the <tt style="background: #ebebeb; font-size: 13px;">kateversion</tt> number of _all_ xml files to be on the safe side?</li>
</ul>

<p>I think these issues first need to be clarified before we can commit this.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R216 Syntax Highlighting</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23034">https://phabricator.kde.org/D23034</a></div></div><br /><div><strong>To: </strong>jpoelen, Framework: Syntax Highlighting, dhaumann, cullmann<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, univerz, LeGast00n, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>