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

Unwanted wrapping of link assignments #4933

Closed
jnqnfe opened this issue Jul 30, 2021 · 3 comments
Closed

Unwanted wrapping of link assignments #4933

jnqnfe opened this issue Jul 30, 2021 · 3 comments

Comments

@jnqnfe
Copy link

jnqnfe commented Jul 30, 2021

I've deliberately defined each link on individual lines, and I've deliberately ignored the typical max line length. rustfmt chooses to not ignore the max line length (debatably not so good) and worse, groups them all up into a single "paragraph".

-//! [`Introspector::get_sink_info_by_name()`]: crate::context::introspect::Introspector::get_sink_info_by_name
-//! [`SinkFlagSet::DECIBEL_VOLUME`]: crate::def::SinkFlagSet::DECIBEL_VOLUME
-//! [`SourceFlagSet::DECIBEL_VOLUME`]: crate::def::SourceFlagSet::DECIBEL_VOLUME
+//! [`Introspector::get_sink_info_by_name()`]:
+//! crate::context::introspect::Introspector::get_sink_info_by_name [`SinkFlagSet::DECIBEL_VOLUME`]:
+//! crate::def::SinkFlagSet::DECIBEL_VOLUME [`SourceFlagSet::DECIBEL_VOLUME`]:
+//! crate::def::SourceFlagSet::DECIBEL_VOLUME

Version string: rustfmt 1.4.37-stable (a178d03 2021-07-26)
Project: https://github.com/jnqnfe/pulse-binding-rust
Specific file: https://github.com/jnqnfe/pulse-binding-rust/blob/master/pulse-binding/src/volume.rs
Ran against the master branch.

@ytmimi
Copy link
Contributor

ytmimi commented Mar 27, 2022

@calebcartwright can we close this? I think it was fixed by the merged PR!

@calebcartwright
Copy link
Member

@calebcartwright can we close this? I think it was fixed by the merged PR!

Indeed, do you have access to close?

@ytmimi
Copy link
Contributor

ytmimi commented Mar 28, 2022

closing as this was fixed by #5096

@ytmimi ytmimi closed this as completed Mar 28, 2022
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

3 participants