nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2024-11-26 00:34:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rollup merge of #113385 - joshtriplett:style-guide-cleanup-chains, r=calebcartwright

Browse Source 
style-guide: Fix chain example to match rustfmt behavior

The style guide gave an example of breaking a multi-line chain element
and all subsequent elements to a new line, but that same example and the
accompanying text also had several chain items stacked on the first
line. rustfmt doesn't do this, except when the rule saying to combine

```
shrt
    .y()
```

into

```
shrt.y()
```

applies.

This is a bugfix to match rustfmt behavior, so it's not a breaking change, and
it just needs a ``@rust-lang/style`` reviewer to r+.
This commit is contained in:
Jubilee 2023-07-11 21:00:28 -07:00 committed by GitHub
commit 3518041a84
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/doc/style-guide/src/expressions.md
View File

@ -451,12 +451,11 @@ foo(
#### Multi-line elements #### Multi-line elements
If any element in a chain is formatted across multiple lines, then that element If any element in a chain is formatted across multiple lines, put that element
and any later elements must be on their own line. Earlier elements may be kept and any later elements on their own lines.
on a single line. E.g.,
```rust ```rust
a.b.c()?.d a.b.c()?
.foo( .foo(
an_expr, an_expr,
another_expr, another_expr,