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

ci: fix asset upload workflow

feat: v2 support for nested tuples w/o spaces

Add some basic tests for `try{}` expressions

They failed to parse in rustfmt on me in https://github.com/rust-lang/rust/pull/77877, which looks like it's since been fixed, but I figured I'd send in some tests anyway.

tests: adjust some cherry-picekd tests

fix: resolve some parser related bugs

tests: add test for panic on new_parser_from_file

fix(parser): better unclosed delims handling

deps: bump rustc-ap to v686

Fix overriding license header bug (#4488)

* Override header to None when path len = 0

* Add regression test

* Ensure nightly

Preserve comments in empty statements (#4180)

* Preserve comments in empty statements

Closes #4018

* fixup! Preserve comments in empty statements

Correctly indent skipped-over code

Closes #4398

deps: bump rustc-ap to v684

feat: add initial support for ConstBlock expressions

deps: update macro parsing

deps: bump rustc-ap to v683

Release v1.4.22

refactor: move env/osstr imports to avoid warnings

Use rustfmt given by RUSTFMT env var (#4419)

fixup! Preserve and format type aliases in extern blocks

fixup! Preserve and format type aliases in extern blocks

fixup! Preserve and format type aliases in extern blocks

Preserve and format type aliases in extern blocks

Previously, non-trivial type aliases in extern blocks were dropped by
rustfmt because only the type alias name would be passed to a rewritter.
This commit fixes that by passing all type information (generics,
bounds, and assignments) to a type alias rewritter, and consolidates
`rewrite_type_alias` and `rewrite_associated_type` as one function.

ci: backport GHA workflows

feat: support config. of leading match arm pipe

docs: add config info for match_arm_leading_pipes

tests: add tests for leading pipe config

Add regression test for unsafe mods

deps: bump rustc-ap to v679

deps: bump rustc-ap to v678

bump rustfmt version to v1.4.21

fix: inner attribute formatting

Fix module resolution in inner modules with paths (#4194)

chore: run rustfmt

deps: bump rustc-ap to v677

deps: bump rustc-ap to v673

deps: bump rustc-ap to v672

tests: add system tests for inverted span issue

fix: inverted span on attrs within closure

deps: bump rustc-ap to v671 (#4356)

preparation for potential rustfmt 1.4.19 (#4283)

chore: backport 8157a3f0afe978d3e953420577f8344db7e905bf

deps: bump rustc-ap to v669

deps: bump rustc-ap-* to v668

deps: bump rustc-ap* to v666

Use correct span for match arms with the leading pipe and attributes (#3975)

Release v1.4.18

Merge pull request #4253 from calebcartwright/rustfmt-1.4.18

bump rustfmt 1.x rustc-ap* crates to v644

fix: backport fix for #4079

fix: backport fix for #4020

deps: bump structopt (and proc-macro-error)

fix: backport fix for submod parser errors

fix: backport mod resolution error handling

Switch to std::error::Error for errors (#3948)

deps: apply upstream rustc-* changes

deps: bump rustc-ap* to v664

Release v1.4.17

Bump parking_lot

Support module-level rustfmt::skip in sub-modules

Release v1.4.15

Cargo update

Update rustc-ap-* crates to 659.0.0 for rustfmt-1.4.15 (#4184)