b4n left a comment (geany/geany#4337)

> Also fixes keywords not being highlighted because of incorrect keyword group 
> name "keywords" instead of "primary" as defined in `highlightmappings.h` 
> (apparently not many people using Geany for Cobol).

Good catch!  Who knew COBOL was a niche language? :laughing: 

> A possible alternative of the patch would be to keep "keywords" in the config 
> file and replace "primary" in `highlightmappings.h`.

Nah, better keep it as it would have worked: not only it's more consistent with 
`secondary`, but also if anybody modified their filetype file to actually work, 
it's better to keep it that way.

PS: I know the name is way too long and confusing, but it's 
*highlight**ing**mappings.h* :wink: 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/4337#issuecomment-2975932832
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/4337/[email protected]>

Reply via email to