stabilise remove_nested_parens and set default to true

This commit is contained in:
Nick Cameron 2018-05-18 16:41:21 +12:00
parent 8afe367510
commit a70f71610c
2 changed files with 17 additions and 17 deletions

View file

@ -1260,7 +1260,7 @@ Convert /* */ comments to // comments where possible
- **Default value**: `false` - **Default value**: `false`
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: Yes - **Stable**: No
#### `false` (default): #### `false` (default):
@ -1286,24 +1286,25 @@ fn adipiscing() -> usize {}
Remove nested parens. Remove nested parens.
- **Defalut value**: `false`, - **Default value**: `true`,
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: No - **Stable**: Yes
#### `false` (default):
```rust
fn main() {
((((foo()))));
}
```
#### `true`: #### `true` (default):
```rust ```rust
fn main() { fn main() {
(foo()); (foo());
} }
``` ```
#### `false`:
```rust
fn main() {
((((foo()))));
}
```
## `reorder_imports` ## `reorder_imports`
@ -1312,7 +1313,7 @@ separated by a newline).
- **Default value**: `true` - **Default value**: `true`
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: No - **Stable**: Yes
#### `true` (default): #### `true` (default):
@ -1339,7 +1340,7 @@ Reorder `mod` declarations alphabetically in group.
- **Default value**: `true` - **Default value**: `true`
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: No - **Stable**: Yes
#### `true` (default) #### `true` (default)
@ -1764,7 +1765,7 @@ Use field initialize shorthand if possible.
- **Default value**: `false` - **Default value**: `false`
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: No - **Stable**: Yes
#### `false` (default): #### `false` (default):
@ -1806,7 +1807,7 @@ Replace uses of the try! macro by the ? shorthand
- **Default value**: `false` - **Default value**: `false`
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: No - **Stable**: Yes
#### `false` (default): #### `false` (default):
@ -1831,7 +1832,7 @@ Break comments to fit on the line
- **Default value**: `false` - **Default value**: `false`
- **Possible values**: `true`, `false` - **Possible values**: `true`, `false`
- **Stable**: Yes - **Stable**: No
#### `false` (default): #### `false` (default):

View file

@ -82,6 +82,7 @@ create_config! {
"Where to put a binary operator when a binary expression goes multiline."; "Where to put a binary operator when a binary expression goes multiline.";
// Misc. // Misc.
remove_nested_parens: bool, true, true, "Remove nested parens.";
remove_blank_lines_at_start_or_end_of_block: bool, true, false, remove_blank_lines_at_start_or_end_of_block: bool, true, false,
"Remove blank lines at start or end of a block"; "Remove blank lines at start or end of a block";
combine_control_expr: bool, true, false, "Combine control expressions with function calls."; combine_control_expr: bool, true, false, "Combine control expressions with function calls.";
@ -105,8 +106,6 @@ create_config! {
"Maximum number of blank lines which can be put between items."; "Maximum number of blank lines which can be put between items.";
blank_lines_lower_bound: usize, 0, false, blank_lines_lower_bound: usize, 0, false,
"Minimum number of blank lines which must be put between items."; "Minimum number of blank lines which must be put between items.";
remove_nested_parens: bool, false, false,
"Remove nested parens.";
// Options that can change the source code beyond whitespace/blocks (somewhat linty things) // Options that can change the source code beyond whitespace/blocks (somewhat linty things)
merge_derives: bool, true, true, "Merge multiple `#[derive(...)]` into a single one"; merge_derives: bool, true, true, "Merge multiple `#[derive(...)]` into a single one";