Implemented rough draft of `check` write mode. (#2539)

* Add rough draft of `check` mode. Not unit tested.

* Added assert-cli; simple test case for `--write-mode=check`

* Lightly documented `check` WriteMode

* wrote clearer phrasing for config::options::WriteMode::Check

* Implemented default for WriteMode where default is Overwrite

* Simplified exit code handling

* updated README.md as per @nrc' comment

* collapsed exit-code handling

* Removed write_mode from Summary, introduced partial option parsing earlier

* Handle write-mode parsing in a slightly better way.

Merge pull request #2632 from phansch/master

Link to official nightly documentation

Link to official nightly documentation

Since https://github.com/rust-lang/rust-central-station/pull/40 has been
merged, we have a stable location for the nightly docs.

Update rustc_ap_syntax

Fix example for reorder_imports (#2628)

Default value changed in #2605

Merge pull request #2624 from sinkuu/dots

Add spaces between consecutive `..` `..=`

Add spaces between consecutive `..` `..=`

update rustc-ap-syntax (#2622)

* update rustc-ap-syntax-100.0.0

Fix tests

Fallout from removing TupField

Update rustc-ap-syntax

Update rustc_ap_syntax

nightly-0.4.2

Fix tests

Merge pull request #2618 from topecongiro/issue-1174

Output xml header and footer only once

Output xml header and footer only once

Merge pull request #2603 from topecongiro/merge-nested-imports

Merge imports

Merge pull request #2617 from topecongiro/issue-2616

Preserve trailing comma on macro call when using mixed layout

Cargo update

Preserve trailing comma on macro call when using mixed layout

Merge pull request #2605 from nrc/import-squelch

Coalesce some of the import options

Review changes

rebasing

Update tests

Squash the various 'reorder imports' option into one

cc #2185

Resolve review comments

Simplify UseTree::has_comment

Merge imports with the same prefix into a single nested import

Add merge_imports config option

Add tests for merge_imports config option

Merge pull request #2612 from peterhuene/suppress-unstable-config-options

Suppress unstable config options by default.

Merge pull request #2613 from dwijnand/patch-1

specify nightly required to install from source

specify nightly required to install from source

when using stable cargo install fails due to #![feature] usage:

    error[E0554]: #![feature] may not be used on the stable release channel
      --> /Users/dnw/.cargo/registry/src/github.com-1ecc6299db9ec823/rustc-ap-serialize-67.0.0/lib.rs:24:1
       |
    24 | #![feature(rustc_private, box_syntax)]
       | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

    error[E0554]: #![feature] may not be used on the stable release channel
      --> /Users/dnw/.cargo/registry/src/github.com-1ecc6299db9ec823/rustc-ap-serialize-67.0.0/lib.rs:25:1
       |
    25 | #![feature(core_intrinsics)]
       | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

    error[E0554]: #![feature] may not be used on the stable release channel
      --> /Users/dnw/.cargo/registry/src/github.com-1ecc6299db9ec823/rustc-ap-serialize-67.0.0/lib.rs:26:1
       |
    26 | #![feature(i128_type)]
       | ^^^^^^^^^^^^^^^^^^^^^^

    error[E0554]: #![feature] may not be used on the stable release channel
      --> /Users/dnw/.cargo/registry/src/github.com-1ecc6299db9ec823/rustc-ap-serialize-67.0.0/lib.rs:27:1
       |
    27 | #![feature(specialization)]
       | ^^^^^^^^^^^^^^^^^^^^^^^^^^^

    error: aborting due to 4 previous errors

    error: Could not compile `rustc-ap-serialize`.
    warning: build failed, waiting for other jobs to finish...
    error: failed to compile `rustfmt-nightly v0.4.1 (file:///d/rustfmt)`, intermediate artifacts can be found at `/d/rustfmt/target`

Suppress unstable config options by default.

This commit suppresses the output of unstable config options by default.

Users can specify the `--unstable-features` option to show the config options
that are unstable.

Fixes #2611.

Try to fix a build regression before it happens

Make `error_on_line_overflow` false by default

And improve docs, the error message, etc.

I think false is the better default since such errors should only occur due to a bug in Rustfmt and therefore most users should not be notified of it happening (although to be clear, it might be a 'bug' which only occurs with pathological input and therefore we won't fix it). The error has proven to be confusing and annoying in the past.

Closes #1080

Reorder config options

cc #1974

Fix test

Merge pull request #2609 from csmoe/update_rustc_syntax

update rustc_ap_syntax

update rustc syntax 91.0.0

Cargo update (#2602)

Update `rustc-ap-syntax` to 89.0.0 and fix up breaking changes.

Merge pull request #2601 from topecongiro/fix-merge-mess

Fix up merge mess

Fix up merge mess

Merge pull request #2600 from i1skn/error_on_unformatted

Fix error_on_unformatted and skip_children override

Merge pull request #2599 from topecongiro/issue-2596

Use multiple lines for imports list which exceeds max width

Merge pull request #2593 from topecongiro/issue-2455

Add reorder_impl_items config option

Fix error_on_unformatted and skip_children override

Currently, error_on_unformatted and skip_children options
specified in the config file are discarded.
This happens because CLI options have a higher priority,
but we coerce an absence of a `bool` option to `false`.
In this scenario, an absence of a `bool` option is indistinguishable
from explicetely set as `false`. We should coerce it to `None` instead,
so it does not override the one in the config file.

Merge pull request #2576 from topecongiro/merge-imports

Use normalized form to format use items

Merge pull request #2589 from topecongiro/issue-2588

Do not indent or unindent inside string literal

Merge pull request #2581 from sinkuu/build_script

Rerun build.rs only when .git/HEAD or CFG_RELEASE_CHANNEL is changed

Merge pull request #2580 from remram44/improve-where_single_line-help

Improve `where_single_line` help

Add trailing comma when using mixed layout with block indent

Put imports list on the next line if it exceeds max width

Add a test for #2569

Do not indent or unindent inside string literal

Closes #2588.

Implement LineClasses

Add tests for #2588

Reorder impl items based on its kind

The ordering is defined as follows:

type < cosnt < macro < method

Items that are same kind will not be reordered: that is, the item
that an user has put first comes first.

Add reorder_impl_items config option

typo (#2598)

* typo

* more typos

add tests for macro!(/* comment */) (#2592)

* add tests

Include instructions to install on nightly (#2590)

Let --dump-default-config default to stdout #2579 (#2586)

Merge pull request #2584 from sinkuu/cleanup

Misc cleanups

Do not turn spaces in the beginning of the file into a newline (#2583)

Fix typo

Use ListItem::has_comment

Remove redudant parens

Return String instead of always returing Cow::Owned

Remove stray comment

Its pointee was removed in 04a6d16c7b68587f8e9fa8a1e6715f80408b023b

Do not collect into a Vec by hand

Remove unnecessary `[..]`s

Use Iterator methods

Use str::repeat

Rerun build.rs only when .git/HEAD or CFG_RELEASE_CHANNEL is changed

Improve where_single_line help

Add test #2574 (#2577)

Cargo fmt

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.

Cargo update (#2575)

Update `rustc-ap-syntax` to 82.0.0.

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