fix clippy::redundant_clones warnings.

Merge pull request #3134 from otavio/issue-3104

Stabilize `edition` configuration option

Stabilize `edition` configuration option

Refs: #3104.

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

Merge pull request #3133 from otavio/rework-readme

README: remove old requirement for Rust 2018 edition use

README: remove old requirement for Rust 2018 edition use

From now on, the `Cargo.toml` is taken into account when triggering
formatting using `cargo fmt`.

It is considered editor's duty to pass the proper `--edition` argument
for `rustfmt` if it is being called manually.

Refs: #3104.
Refs: #3129.

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

Merge pull request #3129 from otavio/issue-3104

cargo-fmt: detect Rust edition in use

Merge pull request #3113 from scampi/issue3105

Fix handling of code that is annotated with rustfmt::skip.

cargo-fmt: Take into account the edition for each target

When formatting the crate, with `cargo fmt`, it parses each target
with the specific Rust edition.

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

keep track of lines which formatting was disabled in order to prevent indentation which would cause code right-shifting

Fix handling of code that is annotated with rustfmt::skip.

A rustfmt::skip'ed block is indented although original lines are
returned. In order to resolve this, the leading whitespaces are trimmed
on each line while retaining the layout; this leaves the skipped code
to be indented as necessary by the caller.

Merge pull request #3130 from JoshBrudnak/master

Added crates to the non-alphabetical treated imports

Remove the source test file

Revert "Use `width_heuristics` in the options docs"

This reverts commit fa3cadf62c11cd8163f08029026a8475ad2b60fd.

Merge pull request #3116 from kellerkindt/patch-1

Add howto for CLion / IntelliJ IDE integration

Added test for crate non-alphabetizing

Add howto for CLion / IntelliJ IDE integration

Treat crates non-alphabetically Fixes: #3118

rustfmt: add support to specify the Rust edition as argument

The new `--edition` command line argument allow the setting of the
desired Rust edition to be used.

Refs: #3104.

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

lib: export config::Edition so it can be used externally

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

config: use 2015 as default Edition

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

Cargo.lock: update `cargo_metadata` to 0.6.1 release

The update is especially important as it adds support for the
`edition` field handling, when parsing `cargo` metadata.

Refs: #3104.

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

rustfmt: avoid duplicating the error handling for `--emit`

There is no need to produce the error message again as it is already
produced by the helper method that does the emit mode validation.

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

Use `width_heuristics` in the options docs

It was using an older name

Merge pull request #3126 from topecongiro/issue-3117

Use correct width when formatting type on local statement

Merge pull request #3114 from matthiaskrgr/typos

fix a few typos found via codespell.

Use correct width when formatting type on local statement

Add a test for #3117

fix a few typos found via codespell.

Merge pull request #3115 from matthiaskrgr/clippy

fix a couple of clippy lint warnings

fix a couple of clippy lint warnings

simplify a comparison with "true"
fn can_be_overflowed: remove unused lifetime
fn rewrite_pairs_one_line: pass "list" by reference (it is not consumed in the function)
fn span_for_token_stream: pass "token_stream" by reference since it is not consumed
use tool lints for clippy suppressions

Merge pull request #3109 from scampi/issue-3038

force a newline after the `if` condition if there is a different indentation level

Update rustc-ap-syntax

Remove NLL feature

And do some refactoring in comments.rs

Closes #3107

Remove various feature flags

Add `rustc-workspace-hack` workaround like RLS/Cargo

Catch parser panic in format_snippet (#3103)

force a newline after the `if` condition if there is a different
indentation level

Merge pull request #3106 from sinkuu/clippy

Clippy and cleanups

Merge pull request #3102 from nrc/arm-guard-newline

 Only put `{` on a newline in a match arm where necessary

Merge pull request #3101 from nrc/pair-newline

Simplify multi-lining binop exprs

Cargo.lock

Reduce allocations

Use concat() instead of join("")

Cleanup

Clippy

Use UnOp::to_string

Remove an unsafe code

try to fix comment bad wrapping (#3099)

Add test (issue 3005)

Only put `{` on a newline in a match arm where necessary

Fixes #3005

add test

Merge pull request #3096 from otavio/use-bytecount-count

utils: rewrite `count_newlines` using `bytecount::count`

Fixup formatting

Simplify situations in which the last sub-expr in a bin-op can go multiline without making the whole expr multiline

Fixes #3034

utils: rewrite `count_newlines` using `bytecount::count`

This uses a optimized byte count and also makes use of SIMD
instructions to optimize the processing of the byte arrays.

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

Merge pull request #3100 from topecongiro/issue-3092

Fix poor formatting of empty trait with generic bounds

Merge pull request #3093 from scampi/issue2633

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

Merge pull request #3091 from topecongiro/issue-3060

Do not add parens around lifetimes

Use correct width

When rewriting trait bounds on the next line, we do not want to add an extra
indentation. However, the max width should be smaller.

Add a correct indent before trait's closing brace with empty body

Add a test for #3092

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