Merge pull request #3095 from otavio/avoid-overflow-control-flow-expr

Avoid control flow expressions conditions to go multi line

Merge pull request #3094 from otavio/avoid-unwrap-or

Replace `.unwrap_or` with `.map_or` in few places

Avoid control flow expressions conditions to go multi line

Extends the multi line condition to over other control flow
expressions, it now covers: `if`, `if let`, `for`, `loop`, `while`,
`while let` and `match`.

Refs: #3029

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

handle lines prefixed with a # inside code blocks

Merge pull request #3090 from otavio/issue-3029

Only combine `match` if its condition expression fits in a single line

bin: replace `.unwrap_or` with `.map_or`

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

config: replace `.unwrap_or` with `.map_or`

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

expr: replace `.unwrap_or` with `.map_or`

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Only combine `match` if its condition expression fits in a single line

This improves the formatting and reading of code avoiding the
condition expression to be rewrite, if it goes multi line.

Fixes: #3029.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

closures: replace `.unwrap_or` with `.map_or`

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

do not vertically align list items in case the tactic is Horizontal

Do not add parens around lifetimes

Parens should be added only around trait objects.

Add a test for #3060

Add `attr_fn_like_width` to the width heuristics

It is a bit larger than than `fn_call_width` which we used previously, so fewer attributes get reformatted.

Closes #2929

Merge pull request #3089 from topecongiro/format-comment

Add format_doc_comments

Merge pull request #3086 from otavio/bump-rustc-ap-272

Update rustc-ap-* to 272.0.0

Add format_doc_comments

Merge pull request #3088 from otavio/fix-misleading-comment

rewrite: adjust comment as it refers to `is_if_else_block`

rewrite: adjust comment as it refers to `is_if_else_block`

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Merge pull request #3087 from scampi/lineoverflow

fixed display message for the LineOverflow error

fixed display message for the LineOverflow error

Update rustc-ap-* to 272.0.0

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Merge pull request #3084 from otavio/issue-1990

Enable `stdin_disable_all_formatting_test` test

Merge pull request #3083 from scampi/itemized_blocks

Handle itemized blocks in comments

rewrite_string: detect when a url is being split and place the new line
after it

Merge pull request #3085 from otavio/extend-gitignore

.gitignore: Ignore Emacs backup files

Enable `stdin_disable_all_formatting_test` test

Fixes: #1990.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

.gitignore: Ignore Emacs backup files

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

handle itemized items inside comments

Merge pull request #3082 from otavio/bump-dependencies

Bump dependencies

Merge pull request #3080 from topecongiro/issue-3031

Format macro calls with item-like arguments

Bump rustc-ap-* to 270.0.0

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Replace `isatty` crate with `atty`

The `isatty` crate has been deprecated and a replacement has been
released for it called `atty`. It offers a nicer API and the code
change to adapt is trivial.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

rewrite_string: allow to break on a boundary character that is on edge

rewrite_string: retain blank lines that are trailing

rewrite_string: take care of blank lines appearing within the last line

rewrite_string: handle newlines in the last line that fits in the shape for cases where line_start is not a whitespace

rewrite_string: do not overflow the next line when the line_start is not a whitespace

rewrite_string: trim trailing whitespace in case the last_line fits in the given shape

Format a macro call with multile item-like arguments

Format a macro call with a single item-like argument

Add a test for #3031 and update test

No combining for an item-like macro argument.

Merge pull request #3078 from YaLTeR/fix-closure-indentation

Fix inconsistent overflow behavior in Visual style

Merge pull request #3077 from YaLTeR/fix-extra-commas

Get rid of extra commas in Visual struct literal formatting

Merge pull request #3070 from topecongiro/issue-3030

Format a paren expr with double slash comment

Merge pull request #3069 from topecongiro/issue-3003

Consider a multi-lined array as a block-like expression

Merge pull request #3067 from topecongiro/refactor-toexpr

Add println!-like heuristic to the fail attribute

Merge pull request #3063 from otavio/add-doc-is_doc_comment

comment: Add documentation to `is_doc_comment` public method

Merge pull request #3079 from topecongiro/rustc-ap

Cargo update

Fix empty types being inserted to closure

cc https://github.com/rust-lang/rust/pull/54229.

Cargo update

Overflow Visual functions even with one argument

Add a test for #3049

No trailing comma in struct literals (Visual)

Add a test for #3066

Merge pull request #3073 from scampi/format_strings

format_strings: take into account newline occurring within a rewritten line

format_strings: take into account newline occurring within a rewritten line

Format a paren expr with double slash comment

Add a test for #3030

Consider a multi-lined array as a block-like expression

Add a test for #3003

and run cargo fmt.

Update .travis.yml

Format and update test

Put arguments of the fail attribute on the same line as println!

Move maybe_get_args_offset to overflow.rs

Avoid using to_expr as much as possible

Get rid of ToExpr trait

Take impl Iterator for overflow routines

comment: Add documentation to `is_doc_comment` public method

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Merge pull request #3061 from mehcode/feature/default-edition

Fix help message for edition config option

Merge pull request #2816 from moxian/enum-discrim

Add option to vertically align enum discriminants.

Fix help message for edition config option

Merge pull request #3050 from andrehjr/empty-impl-body-with-braces-newline

Trait Implementation - Adds a newline before the last brace on impls without a body if the first brace is also on a newline

Merge pull request #3052 from scampi/contributing

added a note about the use of the RUST_LOG environment variable

do not format a code block in documentation if it is annotated with ignore or text (#3058)

added a note about the use of the RUST_LOG environment variable

Change enum align semantics to care about absolute length, rather than difference.

If we're only aligning enum discriminants that are "not too far apart
(length-wise)", then this works really well for enums with
consistently-long or consistently-short idents, but not for the mixed
ones.
However, consistently-long idents is somewhate of an uncommon case and
overlong idents may be allowed to be formatted suboptimally if that
makes mixed-length idents work better (and it does in this case).

Add option to vertically align enum discriminants.

Makes brace behavior consistent with empty bodies for traits and impls

Adds a newline before the last brace on impls without a body if the first brace is also on a newline

Update rustc_ap_syntax

Remove unnecessary feature flag

Merge pull request #3048 from emilio/merged-comments

lists: Detect block comment by starting from the end.

Merge pull request #3045 from scampi/issue2917

Prevent right-shifting of block comments with bare lines.

Merge pull request #3044 from orium/fix-too-many-modsep

Fix bug in import where two consecutive module separators were possible.

lists: Detect block comment by starting from the end.

The issue with the current code is that comments are collapsed, so comments like
the one from the test end up in a string like:

```
"// this is a single line comment\n/* block = */"
```

I chose to fix it by detecting whether we're in a block comment starting from
the end instead, and tested a single-line comment ended in `*/` just for sanity,
ensuring line breaks are not removed in that case, which would break the
formatting.

The right fix eventually is probably to lex the comments properly, but this does
the work for now, I guess :)

Fixes #3025

Prevent right-shifting of block comments with bare lines.

Lines that didn't start with a comment sigil were returned unchanged in
comment::rewrite_comment. Then these unchanged lines were indented in
MacroBranch::rewrite.

Fix bug in import where two consecutive module separators were possible.

Fixes #3043.

Merge pull request #3042 from topecongiro/issue-3040

Do not trim a block from expression if its condition will go multi-line

Do not trim a block from expression if its condition will go multi-line

Add a test for #3040

Merge pull request #3036 from topecongiro/issue-2932

Combine chain items only when the item gets orphaned otherwise

Merge pull request #3035 from topecongiro/issue-3006

Format generics on associated types

Cargo fmt

Fix shape for index

Combine chain items only when the item will get orphaned

Update tests

Format generics on associated types

Add a test for #3006

Merge pull request #3028 from scampi/issue2973

Fix indent computation of a macro with braces.

Merge pull request #3002 from lqd/normalize-doc-attributes

normalize_doc_attributes option: convert doc attributes to comments