[frameworks-syntax-highlighting] [Bug 386827] Errors in syntax definitions

Sun Dec 3 17:40:51 UTC 2017

See also: https://kate-editor.org/2017/12/03/syntax-highlighting-checker/


On Sun, Dec 3, 2017 at 5:41 PM, Dominik Haumann <dhaumann at kde.org> wrote:
> Hi again,
>
> current list looks like this:
>
> Unused keyword lists:
> - "ample.xml" Unused keyword lists: QSet("icprops", "sgfct", "dvafct")
> - "ansforth94.xml" Unused keyword lists: QSet("attention")
> - "css.xml" Unused keyword lists: QSet("mediatypes_op")
> - "dosbat.xml" Unused keyword lists: QSet("not", "else")
> - "euphoria.xml" Unused keyword lists: QSet("constants")
> - "freebasic.xml" Unused keyword lists: QSet("Assembly Operators")
> - "fsharp.xml" Unused keyword lists: QSet("symbols")
> - "ilerpg.xml" Unused keyword lists: QSet("pkeywords", "evalopcodes8")
> - "metafont.xml" Unused keyword lists: QSet("notDefined", "EnvDelimiters")
> - "pango.xml" Unused keyword lists: QSet("endtags", "tags",
> "int_attributes", "plain_attributes", "color_attributes")
> - "pony.xml" Unused keyword lists: QSet("literal", "types")
> - "powershell.xml" Unused keyword lists: QSet("operators", "attributes")
> - "prolog.xml" Unused keyword lists: QSet("lists ISO", "listing
> non-ISO", "directives non-ISO", "lists non-ISO", "terms non-ISO",
> "streams deprecated", "list+is_list non-ISO")
> - "scss.xml" Unused keyword lists: QSet("mediatypes_op")
> - "vhdl.xml" Unused keyword lists: QSet("forOrWhile", "directions")
> - "xonotic-console.xml" Unused keyword lists: QSet("Aliases")
> - "build/frameworks/syntax-highlighting/data/css-php.xml" Unused
> keyword lists: QSet("mediatypes_op")
>
> Unused contexts:
> - "abc.xml" Unused contexts: QSet("Part")
> - "ample.xml" Unused contexts: QSet("AfterHash", "Region Marker", "Outscoped")
> - "ansys.xml" Unused contexts: QSet("functions_arg", "functions")
> - "asterisk.xml" Unused contexts: QSet("Commentar 1")
> - "boo.xml" Unused contexts: QSet("Single A-comment", "Single Q-comment")
> - "cg.xml" Unused contexts: QSet("Commentar/Preprocessor", "Outscoped")
> - "cisco.xml" Unused contexts: QSet("Parameter", "String")
> - "dosbat.xml" Unused contexts: QSet("Assign")
> - "elixir.xml" Unused contexts: QSet("Find closing block brace",
> "Comment Line", "regexpr_rules")
> - "email.xml" Unused contexts: QSet("body-context")
> - "fasm.xml" Unused contexts: QSet("Preprocessor")
> - "ferite.xml" Unused contexts: QSet("unknown 2", "unknown")
> - "fgl-4gl.xml" Unused contexts: QSet("Normal Text 2", "Normal Text 3")
> - "fgl-per.xml" Unused contexts: QSet("Normal Text 2", "Normal Text 3")
> - "ftl.xml" Unused contexts: QSet("comment", "values")
> - "gcc.xml" Unused contexts: QSet("GNUMacros")
> - "haml.xml" Unused contexts: QSet("comment2", "Comment Line",
> "stringx", "string", "comment0")
> - "ilerpg.xml" Unused contexts: QSet("EvalOCCont")
> - "isocpp.xml" Unused contexts: QSet("DetectIdentifierEnd")
> - "jam.xml" Unused contexts: QSet("RuleDefinitionFull")
> - "julia.xml" Unused contexts: QSet("curly", "squared", "nested")
> - "kotlin.xml" Unused contexts: QSet("symbols")
> - "latex.xml" Unused contexts: QSet("ToEndOfLine")
> - "metafont.xml" Unused contexts: QSet("ToEndOfLine")
> - "modula-2.xml" Unused contexts: QSet("Comment3")
> - "nesc.xml" Unused contexts: QSet("Some Context2", "Some Context")
> - "perl.xml" Unused contexts: QSet("package_qualified_blank", "end_handle")
> - "povray.xml" Unused contexts: QSet("Commentar")
> - "protobuf.xml" Unused contexts: QSet("Commentar")
> - "ruby.xml" Unused contexts: QSet("Comment Line")
> - "sisu.xml" Unused contexts: QSet("indent")
> - "stata.xml" Unused contexts: QSet("Comment 2", "Base")
> - "systemverilog.xml" Unused contexts: QSet("Define", "Outscoped")
> - "tcsh.xml" Unused contexts: QSet("SubstCommand", "HereDoc",
> "SubstFile", "AssignSubscr", "ProcessSubst")
> - "varnishtest.xml" Unused contexts:
> QSet("varnish_expectation_arg1_quoted-string",
> "varnish_expectation_arg1_brace-string",
> "varnish_expectation_arg1_unquoted-string")
> - "varnishtest4.xml" Unused contexts:
> QSet("varnish_expectation_arg1_quoted-string",
> "varnish_expectation_arg1_brace-string",
> "varnish_expectation_arg1_unquoted-string")
> - "verilog.xml" Unused contexts: QSet("Some Context2", "Port")
> - "xharbour.xml" Unused contexts: QSet("logic")
> - "xmldebug.xml" Unused contexts: QSet("45:Enumeration or End", "51:unused")
>
> Reference of non-existing keyword list:
> - "asp.xml" Reference of non-existing keyword list: QSet("Others")
> - "qml.xml" Reference of non-existing keyword list: QSet("keywords")
> - "template-toolkit.xml" Reference of non-existing keyword list: QSet("Others")
>
> Reference of non-existing itemData:
> - "haml.xml" Reference of non-existing itemData attributes:
> QSet("Escaped Text", "Ruby embedded in haml", "Array")
> - "lilypond.xml" Reference of non-existing itemData attributes: QSet("Tremolo")
> - "metafont.xml" Reference of non-existing itemData attributes:
> QSet("Tex", "Bullet", "Verbatim")
> - "relaxng.xml" Reference of non-existing itemData attributes:
> QSet("Entity Reference")
> - "rhtml.xml" Reference of non-existing itemData attributes:
> QSet("RUBY RAILS ERB Text")
> - "rmarkdown.xml" Reference of non-existing itemData attributes:
> QSet("Markdown", "Document Headers")
> - "stata.xml" Reference of non-existing itemData attributes: QSet("String2")
>
> Duplicate contexts:
> - "diff.xml" context duplicate: "File"
> - "objectivecpp.xml" context duplicate: "Preprocessor"
>
> Too many chars in DetectChar / Detect2Chars:
> - "xmldebug.xml" line 130 'char' must contain exactly one char: "\"(\\s+|$)"
> - "xmldebug.xml" line 135 'char' must contain exactly one char: "'(\\s+|$)"
> - "xmldebug.xml" line 374 'char' must contain exactly one char: "\"(\\s+|$)"
> - "xmldebug.xml" line 379 'char' must contain exactly one char: "'(\\s+|$)"
> - "xmldebug.xml" line 416 'char' must contain exactly one char: "\"(\\s+|$)"
> - "xmldebug.xml" line 421 'char' must contain exactly one char: "'(\\s+|$)"
> - "xmldebug.xml" line 468 'char' must contain exactly one char: "\"(\\s+|$)"
> - "xmldebug.xml" line 473 'char' must contain exactly one char: "'(\\s+|$)"
>
> Cheers,
> Dominik
>
>
> On Sun, Dec 3, 2017 at 4:27 PM, Dominik Haumann <dhaumann at kde.org> wrote:
>> Hi all,
>>
>> I just added many compile-time checks to the syntax highlighting indexer.
>> As result, compiling the syntax-highlighting framework now raises many
>> warnings. Would be nice if we could get these fixed until next 5.42 release.
>>
>> Greetings
>> Dominik
>>
>>
>> ---------- Forwarded message ----------
>> From: Dominik Haumann <bugzilla_noreply at kde.org>
>> Date: Sun, Dec 3, 2017 at 4:24 PM
>> Subject: [frameworks-syntax-highlighting] [Bug 386827] Errors in
>> syntax definitions
>> To: dhaumann at kde.org
>>
>>
>> https://bugs.kde.org/show_bug.cgi?id=386827
>>
>> Dominik Haumann <dhaumann at kde.org> changed:
>>
>>            What    |Removed                     |Added
>> ----------------------------------------------------------------------------
>>              Status|UNCONFIRMED                 |CONFIRMED
>>      Ever confirmed|0                           |1
>>
>> --- Comment #5 from Dominik Haumann <dhaumann at kde.org> ---
>> The following checks are now in the KSyntaxHighlighting Framework 5.42:
>>
>> - Validate that for all attributes an itemData exists
>>   https://phabricator.kde.org/R216:2b8b664e15c0dd2945458d9373f2324e0c69056e
>>
>> - Highlighting indexer: Check DetectChar and Detect2Chars
>>   https://phabricator.kde.org/R216:9d30439abda876601d8b6bb1ba2785b9d49e88a8
>>
>> - Highlighting Indexer: Check for duplicate itemDatas
>>   https://phabricator.kde.org/R216:6b7cd0c8735aecf543abf54bf8dcabb783ce6b98
>>
>> - Highlighting Indexer: Warn about duplicate contexts
>>   https://phabricator.kde.org/R216:621e282acbcb98d486e61f3d65d3293bf342489d
>>
>> - Highlighting Indexer: Check keyword lists
>>   https://phabricator.kde.org/R216:463dfc78b4be7b11150420c076a8582be8b98607
>>
>> - Try detecting unused contexts
>>   https://phabricator.kde.org/R216:cd7f2837aad12f18b908e7e733a891ed71fdd400
>>
>> That is, the highlighting indexer has a lot of checks now, and currently raises
>> many many warnings. These should be fixed until the next 5.42 release.
>>
>> --
>> You are receiving this mail because:
>> You are watching the assignee of the bug.