@b4n commented on this pull request.


In src/highlighting.c:

> +				/* FIXME: this is sub-stylable, but we can't figure out the base
+				 * style without the Scintilla instance -- and we can't simply
+				 * assume that if the style ID is larger than the max possible
+				 * one it's this, because SCE_C_IDENTIFIER is also sub-stylable... */

This I don't see how to fix short of having a separate API taking a ScintillaObject instead of a lexer ID. We might be able to store info about the allocated sub-style somewhere that could be accessed from here if we could rely on sub-style allocation to give predictable values. This would require Scintilla to actually be deterministic about that, which it might be, but likely not if some calls could happen adding or modifying substyles (think of a plugin that would mess with this).


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3794/review/1946148904@github.com>