Syntax hightlighting rule order bogus

[mc-butler]

Important

This issue was migrated from Trac:

Origin	https://midnight-commander.org/ticket/2323
Reporter	ossi (@ossilator)
Mentions	zaytsev (@zyv)

opening the file makefile.cpp in the editor causes it to be highlighted with Makefile syntax instead of C++ syntax.

the fix is checking the extension rules first.

Note

Original attachments:

• 0001-match-makefile-syntax-towards-the-end.patch (raw) by ossi (@ossilator) on Oct 8, 2020 at 22:29 UTC

[mc-butler]

Changed by ossi (@ossilator) on Aug 26, 2010 at 9:21 UTC (comment 1)

also, the regexp is waaaaay too broad. it should be something like that:

file ([Mm]akefile(\\...\*)?|..\*\\.mk|Kbuild)$ Makefile

[mc-butler]

Changed by zaytsev (@zyv) on Sep 3, 2010 at 12:21 UTC (comment 2)

• Status changed from new to accepted
• Owner set to zaytsev
• Cc set to zaytsev

[mc-butler]

Changed by andrew_b (@aborodin) on Sep 3, 2010 at 15:14 UTC (comment 3)

• Blocked by set to #2308

[mc-butler]

Changed by andrew_b (@aborodin) on Nov 1, 2011 at 6:10 UTC (comment 4)

• Milestone set to Future Releases
• Branch state set to no branch

[mc-butler]

Changed by ossi (@ossilator) on Oct 8, 2020 at 22:29 UTC

• Attachment 0001-match-makefile-syntax-towards-the-end.patch (raw) added

[mc-butler]

Changed by trac on Jan 31, 2025 at 17:21 UTC

• Blocked by #2308 deleted

(In #2308) Ticket #2308 deleted

[mc-butler]

Changed by zaytsev (@zyv) on Jan 31, 2025 at 17:26 UTC (comment 6)

• Summary changed from syntax hightlighting rule order bogus to Syntax hightlighting rule order bogus
• Milestone changed from Future Releases to 4.8.34

TODO:

1. Sort.

1. Fix wrong patterns:

file .\*\\.toml$ TOML\sFile
include toml.syntax

file .\*\\.(mch|ref|imp)$ B\sFile
include b.syntax

2. ..\* -> .\+

3. Patch by ossi.