Run rustfmt

Enable unreachable_pub lint

Merge pull request #3486 from scampi/issue-3197

add document describing rustfmt's stabilisation process

add a condition to the stabilisation process of a option on its design being sound

Merge pull request #3540 from rchaser53/issue-3539

correct to get wrong BytePos

Merge pull request #3541 from davidalber/fix-condition-typo

Fix condition typo

Fix condition typo

correct to get wrong BytePos

Merge pull request #3533 from rchaser53/issue-3532

fix Comma in comment causes no formatting

fix Comma in comment causes no formatting

Merge pull request #3484 from topecongiro/config_derive

Add config_proc_macro

Add doc comment

Use config_type proc macro

Add config_proc_macro

Merge pull request #3530 from xiongmao86/issue2941

Add a test for #2941.

Add a test for #2941.

Release 1.2.2

Merge pull request #3523 from topecongiro/issue-3515

Attempt to format attributes if only they exist

Merge pull request #3522 from topecongiro/issue-3521

Add a path to the parent dir of rustfmt.toml as a prefix to paths in ignore

Suppress spurious parse error messages on skipped files

Add a path to rustfmt.toml to GitIgnore as root path

leave the comment in parentheses of argumentless Fn (#3518)

Disable cargo integration test

Attempt to format attributes if only they exist

Add and update test for #3515

Add a path to the parent dir of rustfmt.toml as a prefix

Paths users specify in `ignore` configuraiton option is relative to the
directory which contains the rustfmt.toml file. When processing the ignore paths
internally, rustfmt should add a path to the directory as a  prefix since
RealPath passed from libsyntax is a full path.

Merge pull request #3520 from rchaser53/improve-gh-page

use filter by hash when first rendering

use filter by hash when first rendering

Merge pull request #3519 from oli-obk/patch-1

"first" element check does nothing

"first" element check does nothing

no need to main the separate wiki page

no need to maintain a separate page for the options given the new configuration browser

Release 1.2.1

Cargo update

Merge pull request #3511 from topecongiro/issue3498

Avoid overflowing item with attributes

Merge pull request #3510 from topecongiro/issue3509

Fix duplication of attributes on a match arm's body

Use annotate-snippets for emitting errors (#3507)

Merge pull request #3512 from scampi/issue-2724

exit integration test successfully if the crate build failed before applying rustfmt

Merge pull request #3506 from rchaser53/issue-3505

fix not to emit version

fix test_ignore_path_set test when run on beta

```
---- ignore_path::test::test_ignore_path_set stdout ----
Warning: can't set `ignore = IgnoreList({"foo.rs", "bar_dir/*"})`, unstable features are only available in nightly channel.
thread 'ignore_path::test::test_ignore_path_set' panicked at 'assertion failed: ignore_path_set.is_match(&FileName::Real(PathBuf::from("src/foo.rs")))', src/ignore_path.rs:51:9
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
failures:
    ignore_path::test::test_ignore_path_set
```

add check for failed tests

exit integration test successfully if the crate build failed before applying rustfmt

The `cargo test --all` command failed and exited the main process with a
SIGINT. Trapping the signal or trying to get the code of a subshell
didn't work.

Close #2724

Avoid overflowing item if it has attributes

Add a test for #3498

not to use Vec

Disallow putting a body with attributes on the same line

Do not include body's attributes in `arrow_span`

Add a test for #3509

Allow specifying glob pattern to ignore config option (#3488)

add the error mapping

fix not to emit version

Merge pull request #3504 from matthiaskrgr/clippy_2

fix a bunch of clippy warnings

fix clippy warnings

clippy::needless_return
clippy::redundant_closure
clippy::or_fun_call
clippy::len_zero
clippy::expect_fun_call
clippy::assertions_on_constants
clippy::identity_conversion
clippy::chars_last_cmp

Merge pull request #3503 from rchaser53/update-readme

update README.md for GitHub page

update README.md for GitHub page

Merge pull request #3495 from scampi/issue-1096

keep missed comments appearing after the struct/enum ident

create GitHub page for Configuration.md (#3485)

Merge pull request #3500 from rchaser53/issue-3499

fix not to delete semicolon for macro with square brackets

fix not to delete semicolon

Merge pull request #3497 from lwshang/master

Fix Travis-CI badge url

Fix Travis-CI badge url

keep missed comments appearing after the struct/enum ident

keep comment appearing between parameter's name and its type (#3491)

add document describing rustfmt's stabilisation process

Merge pull request #3483 from chrisduerr/master

Enable overflow_delimited_expr for structs

Enable overflow_delimited_expr for structs

This fixes https://github.com/rust-lang/rustfmt/issues/3482.

Merge pull request #3480 from sinkuu/cleanup

Cleanups

Merge pull request #3479 from sinkuu/issue_2995

Fix indexing panic on unicode whitespaces

Fix indexing panic on unicode whitespaces

Remove redundant &

Simplify iterators

Write seperator beforehand

Remove a write of empty string

Remove redundant scopes

Use str::repeat

Merge pull request #3472 from devinalvaro/add-print-current-config

Add option to print current config to stdout

Release 1.2.0

Merge pull request #3477 from topecongiro/test-with-larger-stack-size

Test with larger stack size

Remove RUST_MIN_STACK declarations from CI scripts

Run tests in a thread with larger stack size

Add `--print-config current`

Merge pull request #3469 from topecongiro/cargo-fmt

Fix cargo fmt inconsistency

Merge pull request #3470 from topecongiro/issue-3051

Do not add a space after empty impl

Do not add a space after empty impl

Use BTreeMap to guarantee consistent ordering

Make sure that we run rustfmt against every edition

Merge pull request #3467 from topecongiro/issue-3465

Fix bad performance on deeply nested binary expressions

Merge pull request #3459 from scampi/issue-3442

fix line numbering in missed spans and handle file_lines in edge cases

Merge pull request #3468 from topecongiro/refactor-create_config

Move some code out of create_config

Merge pull request #3466 from topecongiro/discard-error-to-sink

Discard error report in silent_emitter

Merge pull request #3454 from rchaser53/issue-3434

add new attribute rustfmt::skip::macros

put rustfmt::skip::macro mention together with the rustfmt::skip bullet point

Move some code out of create_config

Move rust-clippy to allow_failures

Increase stack size on ci

This is required for tests/target/issue-3465.rs.

Avoid rewriting pairs mutiple times

Add a test for #3465

Discard every parsing error in silent_emitter

`EmitterWriter` from rustc is wasting soooo much time constructing an error
message that will never be emitted, instead we just implement our own `Emitter`
that just discards every error message.

add test for visitor mad from same context

reflect scampi review

- fix README.md
- fix BadAttr doc
- is_rustfmt_macro_error => is_unknown_rustfmt_attr

refrect topecongiro reviews

    - &Vec<syntax::ast::PathSegment> => &[ast::PathSegment]
    - remove unnecessary implements
    - transfer skip logic to inside rewrite_macro
    - fix test
    - use util methods in libsyntax
    - use meta_item_list directly
    - avoid no_entry.rs for test using module system
    - add logic to skip rustfmt::skip::macros only
    - remove base_skip_macro_names
    - remove Rc
    - use clone to append skip_macro_names