Preserve standalone carriage returns on windows also

Split apply with auto detection tests into separate fns

Fix auto detection of windows newlines for inputs with multibyte chars

Split auto_detect tests into seperate fns

Extract conversion to windows newlines into fn

Extract literals into constants

Introduce effective newline style

Convert to free standing functions

Extract applying of newline style into own mod

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

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)

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

Release 1.1.1

Merge pull request #3464 from topecongiro/issue-3463

Avoid duplication on the presence of spaces between macro name and !

Avoid duplication on the presence of spaces between macro name and !

Merge pull request #3462 from stjepang/fix-async-formatting

Fix formatting of async blocks

Fix formatting of async blocks

add explanation for rustfmt::skip::macros

use RefCell and Rc

add new attribute rustfmt::skip::macros

add test for function not having attribute

Merge pull request #3460 from scampi/ci-integration-head

show the HEAD of the integration branch to faciliate reproducing an error with rustfmt