Skip to content
This repository has been archived by the owner on Nov 23, 2017. It is now read-only.

Add comment support at end of line #194

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add comment support at end of line #194

wants to merge 2 commits into from

Conversation

cognominal
Copy link

Unlike a whole comment line. A trailing comment does not appear in the
javascript source. That should be fixable but less urgent

Unlike a whole comment line. A trailing comment does not appear in the
javascript source. That should be fixable but less urgent
@cognominal
Copy link
Author

This should fix #127

@cassiemeharry
Copy link
Collaborator

The problem here is that this tosses away the trailing comments entirely. The point of having comments in the grammar at all is so that we can capture them and put them back in the Javascript output.

@cognominal
Copy link
Author

Currently I use this for learning the language. In my code I comment out stuff that does not work yet.
As I said in the commit, this patch is a useful start. Also I think that lines of comments within an statement
or an expression are still problematic.

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

Successfully merging this pull request may close these issues.

2 participants