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

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

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

Merge pull request #3001 from scampi/issue2977

propagate errors about failing to rewrite a macro

Merge pull request #3014 from PSeitz/master

fixes #2914 by handling BadIssue case

Merge pull request #3013 from YaLTeR/fix-issue-2985

Fix wrong shape for last chain items for indent_style = "Visual"

Improve error message when failing cargo metadata (#3024)

Merge pull request #3019 from yangby-cryptape/fix-typo-in-readme

Fix typo in README.md.

Merge pull request #3017 from matthiaskrgr/typo

cargo fmt: fix typo in format_crate(): (verison -> version)

Merge pull request #3012 from YaLTeR/fix-issue-2496

Fix match arm block flattening

Fix indent computation of a macro with braces.

The leading whitespace of a multine string was taken into account when
computing the `min_prefix_space_width`, even if that line couldn't be
trimmed. The consequence is it was always shifting the macro's content
to the right.

Fix typo in README.md.

add test ensuring only doc = "" attributes are normalized to comments

The other shapes of doc attributes shouldn't be normalized or modified.

add tests interleaving doc attrib comments and regular comments

address review comment in Attribute rewrite fn

add non-regression test to existing attributes

when `normalize_doc_attributes` is on, they shouldn't be affected

add non-regression test to existing doc attributes

when `normalize_doc_attributes` is off, they shouldn't be normalized to a doc comment

add test with multiple levels of indents

move config attribute

cargo fmt: fix typo in format_crate(): (verison -> version)

implement Drop from FmtVisitor in order to simplify failures passing

Merge pull request #3000 from dimbleby/dch/typo

Fix typo in usage text

Merge pull request #2998 from ReapOmen/fix-rustfmt-diff-output

Change `print_diff` to output the correct line number.

Merge pull request #3010 from topecongiro/issue-3009

Refactor the corner case of handling long function

Merge pull request #3011 from topecongiro/rustc-ap-syntax

Cargo update

Merge pull request #3015 from YaLTeR/fix-fn-type-comma

Fix extra new line in fn type aliases with indent_style = "Visual"

Don't insert a new line when fn has no return type

Add a visual fn type alias test

fixes #2914 by handling BadIssue case

Fix last chain item shape for Visual indent_style

Add a test for #2985

Use correct heuristic for match block flattening

Add a test for match flattening

Add a test for #2496

Cargo update

Refactor the corner case of handling long function

Add a test for #3009

Merge pull request #2988 from YaLTeR/fix-issue-2922

Use correct indent in rewrite_bare_fn with Visual style

Merge pull request #3007 from alexcrichton/wasm

Support platforms without a timer

Support platforms without a timer

I've dabbled recently in seeing how hard it would be to compile rustfmt to wasm
and then run it in a web browser, and it turns out that it's [not too
hard][wasm]! In addition to patching a few dependencies which already have a
number of patches out rustfmt also needed some modifications to get it to work,
namely avoiding the usage of `Instant::now()` on the "happy path" which doesn't
work on wasm (it just panics).

This commit is an attempt to add a support for this by avoiding using
`Instant::now()` on the wasm target, but panicking if the actual time elapsed is
requested (which doesn't happen unless verbosely logging I believe).

[wasm]: https://alexcrichton.github.io/rustfmt-wasm/

normalize_doc_attributes option: convert doc attributes to comments

Convert `#![doc]` and `#[doc]` attributes to `//!` and `///` doc comments.

propagate errors about failing to rewrite a macro

Fix typo in usage text

Change `print_diff` to output the correct line number.

Merge pull request #2997 from matthiaskrgr/cargo_test_release

fix cargo test --release

fix cargo test --release.

test::verify_check_works was failing in relase mode on my machine.

The problem was it would check for target/debug/rustfmt in release mode instead of
target/release/rustfmt and fail an assert.

This commit fixes it so that cargo check looks for target/debug/rustfmt and
cargo check --release looks for target/release/rustfmt

Merge pull request #2983 from topecongiro/issue-2642

Handle raw string literals in CharClasses

Merge pull request #2993 from topecongiro/rustc-ap

Cargo update

Cargo update

Merge pull request #2992 from max-sixty/clippy2

Clippy 2/2

Merge pull request #2986 from topecongiro/issue-2907

Handle chain w/ try operators with spaces

Merge pull request #2991 from topecongiro/raw-identifier-in-chain

Handle raw identifiers in chain

final clippy changes

Handle raw identifiers in chain