version-gate the trailing semicolon change of return statements inside a match arm

Merge pull request #3232 from ForsakenDurian/patch-1

Fix formatting in Configurations.md

1.0.1

remove  from README.md

Merge pull request #3236 from kngwyu/upd-syntax

Update rustc-ap-syntax to 306.0

Update rustc-ap-syntax to 306.0

Fix formatting in Configurations.md

fixes #3230

Merge pull request #3231 from LucianBuzzo/minor-typo-fixes

Fix minor typos and grammar

Fix minor typos and grammar

Signed-off-by: Lucian <lucian.buzzo@gmail.com>

Merge pull request #3223 from scampi/issue3213

end expressions like return/continue/break with a semicolon

end expressions like return/continue/break with a semicolon

Close #3213

Merge pull request #3222 from scampi/issue-3217

fix the visitor's starting position when visiting a labelled block

fix the visitor's starting position when visiting a labelled block

Close #3217

Merge pull request #3221 from alexreg/cosmetic-1

Cosmetic improvements

Merge pull request #3220 from topecongiro/bytecount/simd-accel

Cargo update

Cosmetic improvements

Cargo update

Update bytecount to 0.4, and use its generic feature.

Closes #3216.

Add a version option

Merge pull request #3215 from scampi/issue3214

allow to run a rustfmt command from cargo-fmt even when there is no target

allow to run a rustfmt command from cargo-fmt even when there is no target

1.0.0

fixup minor bugs

Merge pull request #3177 from kestred/kstenerson/preserve-brace-leading-ws

Format brace-delimited macros with a space after the name

Prefer to break arguments over putting output type on the next line  (#3190)

Always enforce exactly one space between `macro!` and braces (`{}`)

Merge pull request #3205 from matthiaskrgr/clippy

fix a few clippy warnings

Preserve possibly one whitespace for brace macros

fix a few clippy warnings

types.rs:
fix single_char_pattern (use character patters instead of string for .ends_with() and .starts_with()

patterns.rs
expr.rs
file_lines.rs:
fix into_iter_on_ref_ptr (use iter() or iter_mut() instead of into_iter()

tests/mod.rs:
check_files(): take Option<PathBuf> by reference

0.99.9

Update rustc_ap_syntax

test for #3128

Don't align comments on `extern crate`s

Closes #3128

0.99.8

Merge pull request #3200 from scampi/issue-3194

compute the span after a struct-like item based on the ident description

compute the span after a struct-like item based on the ident description

Merge pull request #3189 from scampi/issue3032

fix logic for adding or not a newline after a missed span

fix logic for adding or not a newline after a missed span

Merge pull request #3175 from kestred/kstenerson/delimited-overflow

Add config option to more aggressively allow overflow

Add tests that include comments before the overflow-able params

Replace always true conditionals with `true`

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)

Add docs to Configuration guide

Add config option to more aggressively allow overflow

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>