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

wrap_comments breaks itemized lists #3020

Closed
Ekleog opened this issue Sep 13, 2018 · 2 comments
Closed

wrap_comments breaks itemized lists #3020

Ekleog opened this issue Sep 13, 2018 · 2 comments

Comments

@Ekleog
Copy link

Ekleog commented Sep 13, 2018

Example diff:

 ///
-/// All the parameters ***except for `from_theater`*** should be inserted as sent by the remote
-/// theater, ie. as passed to [`Theater::send`] on the remote actor:
-///  * `from` is the sending (remote) [`ActorId`], as reported by the remote theater by theater-specific means
-///  * `to` is the receiving (local) [`ActorId`], as requested by the remote theater
-///  * `tag` is a tag that identifies the message type
-///  * `msg` is the (serialized) message
+/// All the parameters ***except for `from_theater`*** should be inserted as
+/// sent by the remote theater, ie. as passed to [`Theater::send`] on the
+/// remote actor:  * `from` is the sending (remote) [`ActorId`], as reported by
+/// the remote theater by theater-specific means  * `to` is the receiving
+/// (local) [`ActorId`], as
+/// requested by the remote theater  * `tag` is a tag that identifies the
+/// message type  * `msg` is the (serialized) message
 ///

This currently prevents me from using it to get comments limited to the correct length.

Thank you for your work on rustfmt!

@Ekleog Ekleog changed the title normalize_comments breaks itemized lists wrap_comments breaks itemized lists Sep 13, 2018
@Ekleog
Copy link
Author

Ekleog commented Sep 13, 2018

Whoops, had written normalize_comments instead of wrap_comments in the title, error now fixed.

@Ekleog
Copy link
Author

Ekleog commented Sep 13, 2018

And somehow I missed #627 and found it just after opening this… closing, sorry for the noise!

@Ekleog Ekleog closed this as completed Sep 13, 2018
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

No branches or pull requests

1 participant