Merge pull request #3188 from scampi/issue3124

do not add a newline after a missed span if it is the end of a block comment

Merge pull request #3183 from scampi/issue3182

do not wrap comments in doctest to avoid failing doctest runs

do not add a newline after a missed span if it is the end of a block comment

Cargo update (#3186)

Update `rustc-ap-*` crates to 290.0.0.

Do not count /*/ as both start and end comment (#3185)

Merge pull request #3181 from scampi/issue3131

fix the logic for retaining a comment before the arrow in a match

0.99.7

do not wrap comments in doctest to avoid failing doctest runs

fix the logic for retaining a comment before the arrow in a match

Trim the indentation on macros which heuristically appear to use block-style indentation (#3178)

Merge pull request #3172 from scampi/issue3153

trim the start of a line when wrapping an item in preparation for formatting

Merge pull request #3171 from lqd/whitespace_jam

normalize_doc_attributes: remove whitespace from the doc comment opener

trim the start of a line when wrapping an item in preparation for formatting

normalize_doc_attributes: remove leading whitespace from the doc comment opener

Remove possible whitespace from the `CommentStyle::opener()` so that the literal itself has control over the comment's leading spaces.

This is most useful for tools, such as bindgen, to allow for example: machine-readable comments with precise syntax rules, or idempotently round-tripping between the proc-macro API and rustfmt.

Merge pull request #3165 from scampi/alignment

fix alignment of a struct's fields with the visual style

fix alignment of a struct's fields with the visual style

- rewrite_with_alignment was called from the expr module with the wrong
  shape that missed the extra offset needed for the visual style
- rewrite_with_alignment was indenting the given shape although that
  should have been the caller's responsability

Merge pull request #3145 from otavio/allow-failure-for-crater

CI: only run unit tests for `crater`

Travis CI: Move `futures-rs` out from allow-failures

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

CI: Rework integration script so only lib unit tests are run for crater

Refs: https://github.com/rust-lang-nursery/crater/issues/358

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

Merge pull request #3159 from scampi/issue-3132

The method trim_left_preserve_layout didn't handle tabs properly.

Merge pull request #3152 from topecongiro/do-not-override-edition-in-rustfmt-toml

Use edition in rustfmt.toml when no command line argument is passed

The method trim_left_preserve_layout didn't handle tabs properly.

This is fixed by taking the method macros::indent_macro_snippet which
essentially does the same: it indents a paragraph while preserving the
layout.

Merge pull request #3155 from emilio/missed-spans-bug

missed-spans: Fix bogus check.

missed-spans: Fix bogus check.

I don't really know what it's trying to do, but forgetting about everything
you've seen before when you see whitespace followed by a semicolon doesn't look
right to me, and absolutely no tests were hitting that.

This check was introduced in 5ecdd072d6745cd70ec7c37eea8dbf1f3ee42f2e, however
it was wrong even at that point, and now rustfmt still passes that test,
regardless of macro name.

Fixes #3154.

Merge pull request #3150 from flip1995/patch-1

Remove deprecated cfg_attr from README

Use edition in rustfmt.toml when no command line argument is passed

Remove deprecated cfg_attr from README

`tool_attributes` are stable since 1.30. The old `cfg_attr(rustfmt, rustfmt_skip)` attributes aren't necessary anymore and everyone should switch to `#[rustfmt::skip]` sooner or later.

There is also a Clippy lint in the making for this: rust-lang-nursery/rust-clippy#3123

Merge pull request #3141 from dfyz/issue-2810

Try to fix formatting failures on Windows (issue #2810)

Merge pull request #3144 from otavio/issues-3143

 cargo-fmt: Fix splitting of targets across editions

Merge pull request #3142 from topecongiro/issue-3139

Do not remove path disambiugator inside macro

Fix formatting failures on Windows

When newline_style is set to Windows, an empty line inside of a macro
results in `\r` being passed to the `fold()` in `MacroBranch::rewrite()`.

`\r` is technically not an empty string, so we try to indent it, leaving
trailing whitespaces behind, even though that was not intended
(as far as I can see).

This commit replaces the `!l.is_empty()` check with calling
`is_empty_line()`, since trying to indent any whitespace-only string
will probably result in problematic trailing whitespaces.

Fixes: #2810

cargo-fmt: Fix splitting of targets across editions

When I reworked the code, it ended not generating the complete list of
need targets. Fix it.

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

cargo-fmt: Add --edition when printing the command

By mistake, it was forgotten to print out the edition in use when
printing the rustcmd command. Fix it.

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

Do not remove path disambiugator inside macro

Add a test for #3139

Merge pull request #3140 from matthiaskrgr/redundant_clones

fix clippy::redundant_clones warnings.

fix clippy::redundant_clones warnings.

Merge pull request #3138 from topecongiro/issue-3137

Simplify handling of parens around generic bound trait

Simplify handling parens on generic bound trait

Add a test for #3137

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