<table><tr><td style="">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/D14632">View Revision</a></tr></table><br /><div><div><p>What just came to my mind is that this patch also has an issue: If you include a keyword list of a.xml in b.xml, and a.xml uses different word delimiters for the keyword list, then the keywords in b.xml will not be highlighted/detected correctly.</p>

<p>Did we think about this when approving this change? What solutions do we have for this? The least we can do is to add this i) to the API documentation, and ii) to the highlighting section in the docbook.</p>

<p>That said, the correct fix would be to simply refer to the keyword list of the respective Definition, and if the rule is applied, we should check for the word delimiters of the referenced keyword list...</p>

<p>Any other thoughts?</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/D14632">https://phabricator.kde.org/D14632</a></div></div><br /><div><strong>To: </strong>jpoelen, Framework: Syntax Highlighting, cullmann, dhaumann, vkrause<br /><strong>Cc: </strong>kwrite-devel, vkrause, kde-frameworks-devel, LeGast00n, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>