Skip to content

Add support for nested comments - #9936

Merged
bors merged 1 commit into
rust-lang:masterfrom
madjar:master
Oct 21, 2013
Merged

Add support for nested comments#9936
bors merged 1 commit into
rust-lang:masterfrom
madjar:master

Conversation

@madjar

@madjar madjar commented Oct 18, 2013

Copy link
Copy Markdown
Contributor

This should close #9468.

I removed the test stating that nested comments should not be implemented.

I had a little chicken-and-egg problem because a comment of the std contains "/*", and adding support for nested comment creates a backward incompatibility in that case, so I had to use a dirty hack to get stage1 and stage2 to compile. This part should be revert when this commit lands in a snapshot.

This is my first non-typo contribution, so I'm open to any comment.

@alexcrichton

Copy link
Copy Markdown
Member

Would you mind squashing these commits into one? Also feel free to ping reviewers on the pull request when you update it because sadly github doesn't notify anyone on a push to the branch.

@madjar

madjar commented Oct 20, 2013

Copy link
Copy Markdown
Contributor Author

@alexcrichton I wasn't sure I should do that, thanks for the instructions.

@brson Fixed what you said.

bors added a commit that referenced this pull request Oct 21, 2013
This should close #9468.

I removed the test stating that nested comments should not be implemented.

I had a little chicken-and-egg problem because a comment of the std contains "/*", and adding support for nested comment creates a backward incompatibility in that case, so I had to use a dirty hack to get stage1 and stage2 to compile. This part should be revert when this commit lands in a snapshot.

This is my first non-typo contribution, so I'm open to any comment.
@brson

brson commented Oct 21, 2013

Copy link
Copy Markdown
Contributor

@madjar the main function of the test case you moved to run-pass needs to be pub because of the way all tests are concatenated together on the windows bots.

@madjar

madjar commented Oct 21, 2013

Copy link
Copy Markdown
Contributor Author

@brson Thanks, I wouldn't have guessed it myself. This should be fixed (and squashed).

@alexcrichton

Copy link
Copy Markdown
Member

Hm, it looks like something mahy have gone awry, the test is still just fn main() {}

@madjar

madjar commented Oct 21, 2013

Copy link
Copy Markdown
Contributor Author

Silly me having troubles with git. This is now (really fixed).

bors added a commit that referenced this pull request Oct 21, 2013
This should close #9468.

I removed the test stating that nested comments should not be implemented.

I had a little chicken-and-egg problem because a comment of the std contains "/*", and adding support for nested comment creates a backward incompatibility in that case, so I had to use a dirty hack to get stage1 and stage2 to compile. This part should be revert when this commit lands in a snapshot.

This is my first non-typo contribution, so I'm open to any comment.
@bors bors closed this Oct 21, 2013
@bors
bors merged commit 1dc3d0b into rust-lang:master Oct 21, 2013
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

Successfully merging this pull request may close these issues.

Lex nested block comments

4 participants