Rational: e.g. KTextEditor uses the repository to manage all syntax definitions.
But to allow the user to use "None" as highlighting, we special case that everywhere and use there Definition().
I think it would be much cleaner if the "None" definition is just part of the repository.
An other approach would be to be able to add it later ourself.
Details
Details
With this patch applied, I can just use definitions() everywhere in KTextEditor KateSyntaxManager without any special casing.
Diff Detail
Diff Detail
- Repository
- R216 Syntax Highlighting
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Fixes translation + code editor example.
No special handling needed there (+ fix for section without name)
Comment Actions
Hi, is that change acceptable?
I would need it to continue on my other KTextEditor patches that will not work without this :=)