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

Fix/comments inside trait generics gets duplicated #5446

Conversation

jmj0502
Copy link
Contributor

@jmj0502 jmj0502 commented Jul 16, 2022

Fixes #5358

  • Now the value of comment_hi is equals to the generic.span.lo if there actually are any generic parameters, otherwise it will kept the value previously defined.

@jmj0502 jmj0502 changed the title Fix/comments inside trait generics gets duplicated #5358 Fix/comments inside trait generics gets duplicated Jul 16, 2022
@@ -0,0 +1,4 @@
// Test /* comment */ inside trait generics gets duplicated.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this read Test /* comment */ inside trait generics does not get duplicated?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes! I'll fix it un a few minutes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

Copy link
Contributor

@ytmimi ytmimi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your first contribution 🎉

@ytmimi ytmimi force-pushed the fix/comments-inside-trait-generics-gets-duplicated branch from 4e23b41 to a17dfb2 Compare July 17, 2022 23:22
@ytmimi ytmimi force-pushed the fix/comments-inside-trait-generics-gets-duplicated branch from a17dfb2 to 2ce5697 Compare July 18, 2022 01:11
@ytmimi ytmimi merged commit a7bf009 into rust-lang:master Jul 18, 2022
@ytmimi ytmimi added release-notes Needs an associated changelog entry and removed pr-ready-to-merge labels Jul 18, 2022
@calebcartwright calebcartwright removed the release-notes Needs an associated changelog entry label Jan 24, 2023
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.

/* comment */ inside trait generics gets duplicated
3 participants