Cargo fmt

Format normalized use item

This commit implements `Rewrite` trait on `UseTree`, which is a normalized
form of `ast::UseTree` for rustfmt.

Do not include separator to post comment

This prevents the trailing `;` on use item to be treated as comment.

Do not insert newline when item is empty

This change is necessary when we remove unused imports (`use std::{};`).

Update tests

1. snake_case < CamelCase < UPPER_SNAKE_CASE
2. Use vertical layout for list with nested imports.

Merge pull request #2572 from codeworm96/remove_unreachable

Remove unreachable! from macros.rs

Add test for #2558

When run against invalid macro definitions, rustfmt should leave
them unchanged rather than panic.

Remove unreachable! from macros.rs

replaced unreachable! with error handling using Option.

Closes #2558

Merge pull request #2571 from topecongiro/issue-2569

Avoid panicking on macro call with a single comma

Merge pull request #2557 from topecongiro/vertical-layout-complex-attrs

Use vertical layout for complex attributes

Merge pull request #2562 from topecongiro/issue-2196

Combine simple heuristics for function calls and array

Avoid panicking on macro call with a single comma

`parse_item` from libsyntax may return `None`, so we need to discard
the result in that case.

Update tests

Follow indent style config when formatting attrs

Merge pull request #2563 from rleungx/allow-underscore

allow underscore in macro_rules!

allow underscore

Merge pull request #2568 from mtn/features_typo

/s/featuers/features

/s/featuers/features

Merge pull request #2567 from cramertj/master

Allow stabilization of match_default_bindings

Allow stabilization of match_default_bindings

Merge pull request #2556 from topecongiro/issue-2554

Do not add the beginning vert to the match arm

Update tests and cargo fmt

Format array using overflow module

This commit applies heuristics used for function calls to array
and vice versa.

Factor out default_tactic and fix indentation

rustfmt fails to handle binary expressions with comments in-between.

Replace MacroStyle with ast::DelimToken

Avoid cloning RewriteContext

Use vertical layout for complex attributes

Do not add the beginning vert to the match arm

Pass the span after the match's condition expression.
Closes #2554.

Merge pull request #2549 from topecongiro/macro-def-spaces-around-colon

Add config option to control spaces around colon in macro def

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

Update rustc-ap-syntax to 73.0.0

Cargo fmt and update tests

Remove a space after a colon of metavariable def in macro def

Update a test

Use `UseSegment::Slf` or `UseSegment::Super` when appropriate

Currently we `UseSegment::Ident` for all of the segments except the last.
E.g. `use super::foo::bar::self;` will be
`[Ident("super"), Ident("foo"), Ident("bar"), Self(None)]`.
in the current implementation. I think that this should be
`[Super(None), Ident("foo"), Ident("bar"), Self(None)]`.
instead.

I noticed this because some tests failed after updating
`rustc-ap-syntax` to 73.0.0.

Fix libsyntax updates

`ast::UseTreeKind::Simple` now takes `Option<ast::Ident>`
instead of `ast::Ident`.

Fix libsyntax update

Underscore is now one of keywords.

Cargo update

Update rustc-ap-syntax to 73.0.0.

Merge pull request #2552 from alanhdu/master

Close #2551

Close #2551

Fix build

Merge pull request #2550 from sinkuu/chars_count_index

Don't index a string with chars().count()/position()

Merge pull request #2548 from topecongiro/match-mod

Do not collapse block around expr with condition on match arm

Merge pull request #2535 from nrc/import-ord

Import ordering

Omit unnecessary UTF-8 decoding

Use take_while

Don't index a string with `chars().count()`

Do not collapse block around expr with condition on match arm

Closes #2376.

Add matches module

`matches` module contains `rewrite_match` and related stuffs.

Return grouping `extern crate` to true by default

reviewer comments and rebase fallout

Better handle comments and newlines around erased imports

fixup tests

cargo fmt

Reimplement import reordering.

format code and tests

Reorder imports by default

Merge pull request #2542 from topecongiro/macro-2.0

Handle macro arguments which exceeds max width

Put a space before colon that appears after a meta variable

Closes #2534.

Put spaces around braces

Add some doc comments and factor out add_repeat and add_delimited

Handle binary operators and lifetimes

Break before meta variables when using multiple lines

Remove has_prefix_space

Format macro arguments with vertical layout

Remove unit tests

Update tests

Merge pull request #2541 from topecongiro/issue-2358

Skip name replacement in comments and strings

Merge pull request #2540 from topecongiro/version

Fix print_version

Remove FIXME about duplicated code

Skip name replacement in comments and strings

Add a test for #2538

Fix print_version

0.4.1

Cargo update

Remove `rustc-ap-rustc_errors` from dependencies since it
is re-exported from `rustc-ap-syntax`.

Merge pull request #2509 from topecongiro/issue-2493

Overflow the last rhs of a binary expression

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

Update to the latest rustc-ap-syntax

Call syntax::with_globals before using a parser

Cargo update

Update `rustc-ap-syntax` to `67.0.0`.

Merge branch 'matthew-mcallister-attrib-block-expr'

Format attributes on block expressions

Update CHANGELOG

Merge pull request #2522 from topecongiro/ignore-config-option

Add ignore config option

Merge pull request #2528 from topecongiro/rfc/trait-impl-where

Implement RFC style for trait

Simplify IgnoreList

Simplify join_bounds()

Add a test for #2497

Closes #2497.

Update tests

This is an unintentional side effect of this PR. Nonetheless the diff looks
harmless to me, and it is only relevant when `indent_style = Visual`.
So I think this is ok.

Modify the placement of the opening brace of trait

Put the opening brace on the next line if

1. putting it one the current line exceeds max width.
2. trait bounds uses multiple lines.

Use rewrite_assign_rhs for rewriting bounds

Add rewrite_assign_rhs_with

It is like `rewrite_assign_rhs` but lets us force to put the rhs on the next
line if it uses multiple lines.

This lets us avoid duplicating logic for choosing whether to put stuff on the
same line or the next line.

Update tests for traits with long bounds

The colon should be next to the ident instead of on the next line.

Update tests for braces on trait

Test that the opening brace of trait with long name or bounds with multiple
lines will be put on the next line.

Merge pull request #2513 from rtsuk/master

Restore cargo fmt behavior in workspaces

Merge pull request #2527 from topecongiro/issue-2526

Check whether '\\'' is char literal or lifetime

Merge pull request #2524 from topecongiro/issue-2523

Do not unindent code block in comments with unformattable macro

Merge pull request #2516 from topecongiro/issue-2510

Make rewrite_call_inner more generic

Merge pull request #2529 from dlukes/feat/enable-doctests

Enable doctests

Enable doctests

Doctests were disabled globally because up until #2456, they were just
formatting examples which were not supposed to compile. Now that there
is one runnable doctest, I disabled the other ones individually (by
adding the ignore directive).

I also added some empty lines around the code blocks to avoid the
following warning and instead ignore the code blocks cleanly:

WARNING: ... Code block is not currently run as a test, but will in
future versions of rustdoc. Please ensure this code block is a runnable
test, or use the `ignore` directive.

See rust-lang/rust#28712 for further details.

Merge pull request #2525 from Eijebong/dedup

Dedupe syn/quote

Check whether '\\'' is char literal or lifetime