fix multiline highlighting was ~add ci test for non optional build~

[lievenhey]

trafficstars

This commit adds a ci build to build hotspot without optional dependencies. This should prevents bugs like #575 (fails to build without qcustomplot).

[lievenhey]

I encountered a similar problem when I tried debugging my changes. In that case it was impossible to build hotspot without KSyntaxHighlighter. The ci update should catch that in the future.

[lievenhey]

Nice, the ci found the first bug already

[lievenhey]

Multi line comments are now highlighted correctly

[lievenhey]

dammit, it broke the disassembler

[lievenhey]

now it is working correctly

[GitMensch]

Nice work!

the PR's title may should be adjusted, or the changes split into two PRs (I don't think that's necessary as long as it is split in the commits)

[lievenhey]

I split the ci change off into its own pr

[GitMensch]

I do wonder if not resetting the highlighter has any performance impact

[lievenhey]

It has. Until this patch the Highlighter was called on demand. Now it is called once for the whole file.

[GitMensch]

I guess that's good to go after solving the merge conflict.

[GitMensch]

Just wondering: Is there anything blocking the pull here?

[lievenhey]

Milians high workload.