deps: bump rustc-ap crates to v709

refactor: apply rustc mod parsing changes

chore: apply pattern parsing changes

chore: add span creation util function

deps: bump rustc-ap crates to v708

Fix a `legacy_derive_helpers` warning

Fix some clippy warnings

This commit can be replicated with
`cargo clippy --fix -Z unstable-options && cargo +nightly-2021-02-10 fmt`.

Add the use of rewrite_assign_rhs_with_comments to 1.x

Fix for issue 4603 about extra macro body indentation (third version)

Fix warnings

- Fix nightly warning about `format!`
- Remove unused functions and fields

Bump rustc-ap-* crates to version 706

meta: bump to v1.4.36

deps: apply rustc-ap v705 changes to itemkind variants

chore: bump rust toolchain version

deps: bump rustc-ap crates to v705

meta: bump version to v1.4.35

chore: bump toolchain file

deps: bump rustc-ap to v702

deps: bump rustc-ap crates to v701.0.0

Add arch name to archive files created in the upload workflow (#4679)

* Add arch name to archive files created in the upload workflow

* Fix arch names in matrix.build key to include arch name as well

meta: bump to v1.4.34

fix: don't insert trailing comma on struct lit rest in mac def

tests: add case for issue 4675

meta: bump to v1.4.33

chore: backport some empty block check fixes

refactor: cleanup block check for statements

Fixed semicolon getting moved into comment (fixes #4646)

Added 4646 test case

Add imports_granularity="Item".

This option splits all imports into their own `use` statement.

fix: Avoid incorrect global 'cfg_if' Symbol interning

Fixes #4656

chore: update Changelog with const generic backports

Include const generic type bounds in their spans

Closes #4310

Do not duplicate const keyword on parameters

Use the span after generics and where clause (#4208)

Include constness in impl blocks (#4215)

Closes #4084

Tweak imports_granularity documentation

Disable deprecated_option_merge_imports tests on non-nightly

Rename `merge_imports` to `imports_granularity` and add a `Module` option.

This renames the existing `true`/`false` options to `Crate`/`Never`, then adds a
new `Module` option which causes imports to be grouped together by their
originating module.

meta: bump version v1.4.32

meta: update license years

refactor: remove unneeded clone

fix: indentation issue on generic bounds

meta: bump to v1.4.31

ci: add mingw to path for asset upload job

fix: maintain redundant semis on items in statement pos

Update Cargo.lock.

Add 2021 test.

Fix expected macro formatting test output.

Account for new ast::GenericParamKind::Const::default in rust_ast.

Bump rustc-ap to v697.

Fixes for new rustc changes.

Add support for edition 2021.

ci: fix linking issue  on windows gnu jobs

meta: bump to v1.4.30

Format error and typeof types (#4416)

* Add a test for #4357

* Format error and typeof types

Fix rewrite of closures with a return type

If the closure's body fits in a line, the block is removed but it is
necessary if the closure has a return type.

fix: don't strip nonexistent comma in derive

meta: bump to v1.4.29

Add negative impl test

Preserve polarity on negative non-trait impl

meta: bump to v1.4.28

Comment between typebounds (#4474)

* Test cases and get spans

* Fixed type bounds

* Fixed issue of test cases

* Fixed first test case issue

* Removed unwanted whitespaces

* Removed tmp files

fix: backport some imports for cherry-picked commit

Fixed comment dropped between & and type issue (#4482)

* Fixed comment dropped between & and type issue

* Reduced nesting levels and avoided duplications

* Removed extra allocations

Fixed 'Comment removed between type name and =' issue (#4448)

* Fixed Comment removed between type name and = issue

* Fixed where clause issue and pass the full span

* has_where condition inline

* Fixed indentation error on where clause

* Removed tmp file

fix: apply rustc-ap updates to backported commit

Pick up comments between visibility modifier and item name (#4239)

* Pick up comments between visibility modifier and item name

I don't think this hurts to fix. #2781, which surfaced this issue, has
a number of comments relating to similar but slightly different issues
(i.e. dropped comments in other places). I can mark #2781 as closed and
then will open new issues for the comments that are not already resolved
or tracked.

Closes #2781

* fixup! Pick up comments between visibility modifier and item name

* fixup! Pick up comments between visibility modifier and item name

tests: backport an additional test case

tests

fixes #4115, #4029, #3898

Compare code block line indentation with config whitespace (#4166)

Previously the indetation of a line was compared with the configured
number of spaces per tab, which could cause lines that were formatted
with hard tabs not to be recognized as indented ("\t".len() < "    ".len()).

Closes #4152

Remove `async fn` from extern block tests

Don't drop blocks on foreign functions

A code like

```rust
extern "C" {
    fn f() {
        fn g() {}
    }
}
```

is incorrect and does not compile. Today rustfmt formats this in a way
that is correct:

```rust
extern "C" {
    fn f();
}
```

But this loses information, and doesn't have to be done because we know
the content of the block if it is present. During development I don't
think rustfmt should drop the block in this context.

Closes #4313

Properly format function signature in extern blocks

Closes #4288

And we get to drop a method, which I think is a win :)

Add a test

chore: run rustfmt against source

feat: support underscore expressions

refactor: update cfg_if attr parsing

feat: support struct/slice destructuring

deps: minor rustc_ast changes for v691

deps: bump rustc-ap crates to v691

deps: bump rustc-ap to v687

fix: don't force a newline after an empty where clause

Fixes #4547.

Option to create groups for std, external crates, and other imports

Backport of 17d90ca.

meta: bump to v1.4.27

ci: update GHA workflows

fix: don't drop leading comments in extern

meta: bump version to v1.4.26

ci: restrict GHA workflow triggers on push

tests: attributed comment-only blocks

Fixed 'Incorrect comment indent inside if/else' issue. (#4459)

* Added test cases

* Fixed if condition comment issue

* Fixed extern C issue

* Removed previous test case

* Removed tmp file

* honor the authors intent

* Changed the file name to its original name

* Removed extra whitespace

Correctly create artificial span for formatting closure body

This commit partially reverts #3934, opting to create a span that covers
the entire body of a closure when formatting a closure body with a
block-formatting strategy, rather than having the block-formatting code
determine if the visitor pointer should be rewound. The problem with
rewinding the visitor pointer is it may be incorrect for other (i.e.
non-artificial) AST nodes, as in the case of #4382.

Closes #4382

fixup! Correctly create artificial span for formatting closure body

meta: release v1.4.25

feat: don't insert semi in macro_rules arm body

meta: release v1.4.24

fix: remove ignored depr attribute which now errors

fix: remove comment from cherry-pick that v1.x doesn't handle

Don't flatten a block containing a single macro call

We no longer flatten a block that looks like this:

```rust
match val {
    pat => { macro_call!() }
}
```

Currently, rust ignores trailing semicolons in macro expansion in
expression position (see https://github.com/rust-lang/rust/issues/33953)

If this is changed, flattening a block with a macro call may break the
user's code - the trailing semicolon will no longer parse if the macro
call occurs immediately on the right-hand side of the match arm
(e.g. `pat => macro_call!()`)

meta: bump rustfmt version to 1.4.23