-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Adds support to change the line comment prefix #53
Adds support to change the line comment prefix #53
Conversation
Which assembler does use '#'? |
For me the change does not work. I mean the asm.json is working. |
AT&T syntax for i386. At least the one we use at University But i will look into it more to make it work on all platforms! |
It looks like the npm preLaunchTask that compiles the files are commented out, and I might suspect that the files may have not been compiled when you tested it. I uncommented that line with the latest commit. |
O, I misunderstood. |
I have now merged your PR. |
please have a look here: Seems that # is used also for hex numbers. I think I will change the # grammar so that you need to enter a whitespace after the #, e.g. |
Yes, I was not aware of that. Good catch! This should be fixed. |
Hi, I forgot to ask that. Can you add a statement in the comments that your code is MIT licensed. |
Not sure i understand. Where should i put this statement? |
Just here. Re-assuring that I can use your code :-) |
Ah understand! Yes, you can definitely use my code, and it is MIT licensed :) |
Thanks |
This pull request adds an extension setting that enables the user to change which symbol(s) to use as a line comment prefix. It also adds highlighting for '#' by default. The default prefix is still ';' as to not break any configurations by previous users of the extension.