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.

Sign up for GitHub

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

Jump to bottom

Comment Multiple Lines #100

Open
juuwong opened this issue Feb 3, 2021 · 4 comments · May be fixed by #203
Open

Comment Multiple Lines #100

juuwong opened this issue Feb 3, 2021 · 4 comments · May be fixed by #203

Comments

@juuwong
Copy link

juuwong commented Feb 3, 2021

Is there a way to comment multiple lines in rars 1.5? (i.e. in C we use block comments with /* */ notation)
@TheThirdOne
Copy link
Owner

There is not a way to easily comment multiple lines. You must comment out each line with #.

It might be possible to make RARS accept C style comments without too much effort, but I personally don't plan on implementing it. If you would like to give it a shot, the most relevant file is

rars/src/rars/assembler/Tokenizer.java

Line 290 in 345c17b

while (linePos < line.length) {

I have also considered completely rewriting the assembler; if I do, I will keep C-style multi-line comments in mind.

@TheThirdOne TheThirdOne mentioned this issue Dec 8, 2021
Open
@NilAtabey
Copy link

Please add this

@sdasda7777
Copy link

sdasda7777 commented Oct 18, 2023 •
edited

I'm currently working on adding this feature. I expect to be finished today, stay tuned.

Edit: Didn't manage to finish it, but I think I'm close.

@sdasda7777 sdasda7777 linked a pull request Oct 19, 2023 that will close this issue
Open
@sdasda7777
Copy link

Feel free to check out the Pull Request at #203 and to give me your feedback 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Multiline comment support sdasda7777/rars
4 participants
@TheThirdOne @sdasda7777 @juuwong @NilAtabey