Include trailing commas in wrapped function declarations [RustDoc]

src/librustdoc/html/format.rs

@@ -1438,13 +1438,9 @@ impl clean::FnDecl {
  {
  write!(f, "\n{}", Indent(n + 4))?;
  }
+
+ let last_input_index = self.inputs.values.len().checked_sub(1);
  for (i, input) in self.inputs.values.iter().enumerate() {
- if i > 0 {
- match line_wrapping_indent {
- None => write!(f, ", ")?,
- Some(n) => write!(f, ",\n{}", Indent(n + 4))?,
- };
- }
  if let Some(selfty) = input.to_self() {
  match selfty {
  clean::SelfValue => {
@@ -1477,18 +1473,25 @@ impl clean::FnDecl {
  write!(f, "{}: ", input.name)?;
  input.type_.print(cx).fmt(f)?;
  }
+ match (line_wrapping_indent, last_input_index) {
+ (_, None) => (),
+ (None, Some(last_i)) if i != last_i => write!(f, ", ")?,
+ (None, Some(_)) => (),
+ (Some(n), Some(last_i)) if i != last_i => write!(f, ",\n{}", Indent(n + 4))?,
+ (Some(_), Some(_)) => write!(f, ",\n")?,
+ }
  }
 
  if self.c_variadic {
  match line_wrapping_indent {
  None => write!(f, ", ...")?,
- Some(n) => write!(f, "\n{}...", Indent(n + 4))?,
+ Some(n) => write!(f, "{}...\n", Indent(n + 4))?,
  };
  }
 
  match line_wrapping_indent {
  None => write!(f, ")")?,
- Some(n) => write!(f, "\n{})", Indent(n))?,
+ Some(n) => write!(f, "{})", Indent(n))?,
  };
 
  self.print_output(cx).fmt(f)

tests/rustdoc/async-fn.rs

@@ -79,7 +79,7 @@ struct AsyncFdReadyGuard<'a, T> { x: &'a T }
 
 impl Foo {
  // @has async_fn/struct.Foo.html
- // @has - '//*[@class="method"]' 'pub async fn complicated_lifetimes( &self, context: &impl Bar ) -> impl Iterator<Item = &usize>'
+ // @has - '//*[@class="method"]' 'pub async fn complicated_lifetimes( &self, context: &impl Bar, ) -> impl Iterator<Item = &usize>'
  pub async fn complicated_lifetimes(&self, context: &impl Bar) -> impl Iterator<Item = &usize> {
  [0].iter()
  }

tests/rustdoc/decl-line-wrapping-empty-arg-list.decl.html

@@ -1,2 +1 @@
-<pre class="rust item-decl"><code>pub fn create(
-) -&gt; <a class="struct" href="struct.Padding00000000000000000000000000000000000000000000000000000000000000000000000000000000.html" title="struct decl_line_wrapping_empty_arg_list::Padding00000000000000000000000000000000000000000000000000000000000000000000000000000000">Padding00000000000000000000000000000000000000000000000000000000000000000000000000000000</a></code></pre>
+<pre class="rust item-decl"><code>pub fn create() -&gt; <a class="struct" href="struct.Padding00000000000000000000000000000000000000000000000000000000000000000000000000000000.html" title="struct decl_line_wrapping_empty_arg_list::Padding00000000000000000000000000000000000000000000000000000000000000000000000000000000">Padding00000000000000000000000000000000000000000000000000000000000000000000000000000000</a></code></pre>

tests/rustdoc/decl-trailing-whitespace.declaration.html

@@ -3,7 +3,7 @@
  fn <a href="#tymethod.poll_write" class="fn">poll_write</a>(
  self,
  cx: &amp;mut <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="struct" href="{{channel}}/alloc/string/struct.String.html" title="struct alloc::string::String">String</a>&gt;,
- buf: &amp;mut [<a class="primitive" href="{{channel}}/std/primitive.usize.html">usize</a>]
+ buf: &amp;mut [<a class="primitive" href="{{channel}}/std/primitive.usize.html">usize</a>],
  ) -&gt; <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="enum" href="{{channel}}/core/result/enum.Result.html" title="enum core::result::Result">Result</a>&lt;<a class="primitive" href="{{channel}}/std/primitive.usize.html">usize</a>, <a class="struct" href="struct.Error.html" title="struct foo::Error">Error</a>&gt;&gt;;
 <span class="item-spacer" /> fn <a href="#tymethod.poll_flush" class="fn">poll_flush</a>(self, cx: &amp;mut <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="struct" href="{{channel}}/alloc/string/struct.String.html" title="struct alloc::string::String">String</a>&gt;) -&gt; <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="enum" href="{{channel}}/core/result/enum.Result.html" title="enum core::result::Result">Result</a>&lt;<a class="primitive" href="{{channel}}/std/primitive.unit.html">()</a>, <a class="struct" href="struct.Error.html" title="struct foo::Error">Error</a>&gt;&gt;;
 <span class="item-spacer" /> fn <a href="#tymethod.poll_close" class="fn">poll_close</a>(self, cx: &amp;mut <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="struct" href="{{channel}}/alloc/string/struct.String.html" title="struct alloc::string::String">String</a>&gt;) -&gt; <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="enum" href="{{channel}}/core/result/enum.Result.html" title="enum core::result::Result">Result</a>&lt;<a class="primitive" href="{{channel}}/std/primitive.unit.html">()</a>, <a class="struct" href="struct.Error.html" title="struct foo::Error">Error</a>&gt;&gt;;
@@ -12,6 +12,6 @@
  fn <a href="#method.poll_write_vectored" class="fn">poll_write_vectored</a>(
  self,
  cx: &amp;mut <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="struct" href="{{channel}}/alloc/string/struct.String.html" title="struct alloc::string::String">String</a>&gt;,
- bufs: &amp;[<a class="primitive" href="{{channel}}/std/primitive.usize.html">usize</a>]
+ bufs: &amp;[<a class="primitive" href="{{channel}}/std/primitive.usize.html">usize</a>],
  ) -&gt; <a class="enum" href="{{channel}}/core/option/enum.Option.html" title="enum core::option::Option">Option</a>&lt;<a class="enum" href="{{channel}}/core/result/enum.Result.html" title="enum core::result::Result">Result</a>&lt;<a class="primitive" href="{{channel}}/std/primitive.usize.html">usize</a>, <a class="struct" href="struct.Error.html" title="struct foo::Error">Error</a>&gt;&gt; { ... }
 }</code>

tests/rustdoc/inline_cross/impl_trait.rs

@@ -11,7 +11,7 @@ pub use impl_trait_aux::func;
 // @has impl_trait/fn.func2.html
 // @has - '//pre[@class="rust item-decl"]' "func2<T>("
 // @has - '//pre[@class="rust item-decl"]' "_x: impl Deref<Target = Option<T>> + Iterator<Item = T>,"
-// @has - '//pre[@class="rust item-decl"]' "_y: impl Iterator<Item = u8> )"
+// @has - '//pre[@class="rust item-decl"]' "_y: impl Iterator<Item = u8>, )"
 // @!has - '//pre[@class="rust item-decl"]' 'where'
 pub use impl_trait_aux::func2;
 

tests/rustdoc/line-breaks.rs

@@ -6,7 +6,7 @@ use std::ops::Add;
 // @matches foo/fn.function_with_a_really_long_name.html '//*[@class="rust item-decl"]//code' "\
 // function_with_a_really_long_name\(\n\
 // \ parameter_one: i32,\n\
-// \ parameter_two: i32\n\
+// \ parameter_two: i32,\n\
 // \) -> Option<i32>$"
 pub fn function_with_a_really_long_name(parameter_one: i32, parameter_two: i32) -> Option<i32> {
  Some(parameter_one + parameter_two)

tests/rustdoc/reexports-priv.rs

@@ -98,7 +98,7 @@ pub mod outer {
  pub use reexports::foo;
  // @has 'foo/outer/inner/fn.foo_crate.html' '//pre[@class="rust item-decl"]' 'pub(crate) fn foo_crate()'
  pub(crate) use reexports::foo_crate;
- // @has 'foo/outer/inner/fn.foo_super.html' '//pre[@class="rust item-decl"]' 'pub(in outer) fn foo_super( )'
+ // @has 'foo/outer/inner/fn.foo_super.html' '//pre[@class="rust item-decl"]' 'pub(in outer) fn foo_super()'
  pub(super) use::reexports::foo_super;
  // @!has 'foo/outer/inner/fn.foo_self.html'
  pub(self) use reexports::foo_self;