Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

issue with quoted comment marker #14

Open
GitMensch opened this issue Sep 29, 2020 · 7 comments
Open

issue with quoted comment marker #14

GitMensch opened this issue Sep 29, 2020 · 7 comments

Comments

@GitMensch
Copy link
Collaborator

Some entries actual need an empty "on error do nothing" to override the default behavior (likely of stopping with an error).

A good sample is likely:

PKG_CHECK_MODULES([CJSON], [libcjson], [], [#])

this extension does not stop the inline comment and therefore assumes the macro parameters are not correctly closed, which makes editing all following lines bad (full of red errors and no option to match brackets correct)

@GitMensch
Copy link
Collaborator Author

Just wanted to note that this is still bugging me, I've tried to have a "more or less quick" look at the definition but neither "comment" nor "#" are good search criterias in the tmLanguage definition for autoconf. Any hint where to look at?
I think ] will always end a line # comment, but I have no clue where to add that rule.

@maelvls
Copy link
Owner

maelvls commented Jan 22, 2021

So sorry I didn't get to come back to you before :(

Could you try to open a PR with the change? I would greatly appreciate it!

@GitMensch
Copy link
Collaborator Author

GitMensch commented Jan 22, 2021 via email

Copy link
Owner

maelvls commented Jan 26, 2021

No problem! I added a bit of help in the readme if you want to check it out 😀

@maelvls
Copy link
Owner

maelvls commented Jan 26, 2021

Hmmm I think I know where the bug comes from... it does not seem to be due to the grammar itself (Autoconf.YAML-tmLanguage). I would guess it comes from how VSCode parses comments:

"comments": {
// symbol used for single line comment. Remove this entry if your language does not support line comments
"lineComment": "#"
},

@maelvls
Copy link
Owner

maelvls commented Jan 26, 2021

Nope, doesn't look like an issue with language-configuration.json...

Works:

PKG_CHECK_MODULES([CJSON], [#])

Breaks:

PKG_CHECK_MODULES([CJSON], [], [#])

I'm clueless 🙁

@GitMensch
Copy link
Collaborator Author

Is this an upstream issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants