You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
is there any advance on this issue? because not only the highlighting is broken, but also the conditional declarations highlighting is broken, the disabled/inactive part of the statement should be greyed out, currently we cannot see what is active and inactive which is quite a pain when you have complex makefiles with thousand of lines and hundreds and inactive blocks
In the below example _SUPPORT_64M_MEM is defined, yet the highlighting is wrong and broken
@0x0fe Makefile syntax highlighting is static, it doesn't analyze context, it's how it's implemented.
By "static" do you imply it will only parse the current file and not take into account any other file of the project to compute the synthax highlighting? If so i dont think that is how it should behave and it is definitely not how most IDE behave, most IDE will take into account both all the makefiles and also all the includes, and highligh the makefile accordingly.
Activity
fadeevab commentedon Aug 29, 2017
Fixed here (pull request is still not merged to the master): #15
andrey-gvrd commentedon Oct 6, 2021
This seems still broken.

Sunbreak commentedon Jul 7, 2022
It has been broken for a year since announcement of Microsoft https://devblogs.microsoft.com/cppblog/now-announcing-makefile-support-in-visual-studio-code/
0x0fe commentedon Jan 6, 2023
is there any advance on this issue? because not only the highlighting is broken, but also the conditional declarations highlighting is broken, the disabled/inactive part of the statement should be greyed out, currently we cannot see what is active and inactive which is quite a pain when you have complex makefiles with thousand of lines and hundreds and inactive blocks
In the below example _SUPPORT_64M_MEM is defined, yet the highlighting is wrong and broken
fadeevab commentedon Jan 27, 2023
@0x0fe Makefile syntax highlighting is static, it doesn't analyze context, it's how it's implemented.
@ds0nt @andrey-gvrd @Sunbreak
else-ifeqwill be highlighted starting from the next VSCode release. Fix is here: fadeevab@1d4c0b5Somehow I didn't notice that original complaint under that huge screenshot :)
0x0fe commentedon Feb 12, 2023
By "static" do you imply it will only parse the current file and not take into account any other file of the project to compute the synthax highlighting? If so i dont think that is how it should behave and it is definitely not how most IDE behave, most IDE will take into account both all the makefiles and also all the includes, and highligh the makefile accordingly.