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

Macro on new line ignore comma absence on the previous line #67

Open
bilyushonak opened this issue Jul 21, 2012 · 3 comments
Open

Macro on new line ignore comma absence on the previous line #67

bilyushonak opened this issue Jul 21, 2012 · 3 comments

Comments

@bilyushonak
Copy link

The erlang editor compile the following code:
?assertEqual(aggregated, Response),
?assert(not is_process_alive(TradePid)),
as well as the following (without the comma):
?assertEqual(aggregated, Response)
?assert(not is_process_alive(TradePid)),
without any warning or error

I propose to show warning in this case

@vladdu
Copy link
Collaborator

vladdu commented Jul 21, 2012

Thanks for the suggestion.

This will have to wait a while, because we are rewriting the parser and I'm not sure how to make it fit until the parser is more stable.

@vladdu
Copy link
Collaborator

vladdu commented Jul 21, 2012

@bilyushonak
Copy link
Author

Vlad, thank you for the information and your time

@vladdu vladdu removed the assembla label Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants