Avoid using pretty printer if possible

Setting a pretty printer adds noticeable overhead.

We can replace the usage in `ast:Lifetime::rewrite` by simply converting `Ident`
to string.
We can do the same thing for a macro path as long as it is not nested, which
should hold for most cases.

Do not inspect code for issues or todos when not required

Use debug_assert_eq! to avoid comparing PathBuf on release

Comparing `PathBuf` adds noticeable overhead, avoid it in release build.

Replace lookup_char_pos with lookup_line

`lookup_char_pos` wastes time on fetching unused information. We only need a
`FileMap` and line numbers, so `lookup_line` is enough.

Merge pull request #2465 from karyon/fix_2346

Fix #2346: empty comments should not have any trailing space added

Merge pull request #2461 from topecongiro/reorder-mod

Refactoring: add `reorder` modules

Merge pull request #2464 from topecongiro/beginning-vert

Format a match arm with the beginning vertical bar

Fix #2346: empty comments should not have any trailing space added

Format a match arm with the beginning vertical bar

Merge pull request #2463 from Razican/typo_fix

Fixed small typo in configuration help

Fixed small typo in configuration help

Add TODO

Get rid of macro

Rename fns

Get rid of format_imports

Move reorder-related stuffs in visitor mod to reorder mod

Move rewrite_mod to the items module

Add a reorder module

Merge pull request #2460 from matklad/simple-ci

Simplify CI exaple in the readme

Simplify CI exaple in the readme

rustfmt-preview is now in stable

Merge pull request #2421 from brotzeit/clippy

apply some clippy suggestions

Merge pull request #2454 from radix/issue-2446

Don't generate V: { f: u8, // comment }

only wrap on trailing comments, allow /* */ comments in-line

use contains_comment instead of an ad-hoc check for "//"

hacky solution that makes the tests pass but should really be better

test case for issue #2446

Merge pull request #2453 from muja/patch-1

Fix typo "carfo" -> "cargo" in changelog.

Fix typo "carfo" -> "cargo" in changelog.

Merge pull request #2450 from topecongiro/issue-37

Preserve trailing spaces in doc comments even when options are set

Preserve trailing spaces in doc comments even when options are set

Closes #37.

Merge pull request #2448 from jonathanmorley/patch-1

Rust 1.24 released, rustfmt now works on stable

Merge pull request #2443 from topecongiro/skip-children

Do not try to parse external modules when skipping children

Merge pull request #2436 from topecongiro/markdown-spaces

Preserve trailing two whitespace in comments

Merge pull request #2449 from radix/patch-1

IRC channel is now #rust-dev-tools

IRC channel is now #rust-dev-tools

Also specifically mention the IRC network (irc.mozilla.org) that the channel is on.

Rust 1.24 released, rustfmt now works on stable

Do not try to parse external modules when skipping children

Add rustfmt_skip to a test with long string

Preserve two or more trailing spaces in doc comment

rustdoc treats two or more trailing spaces as a line break.

Preserve trailing whitespaces only for doc comment

Merge pull request #2086 from jugglerchris/difflines_mode

Add a new "modified lines" write mode.

Merge pull request #2422 from davidalber/travis-config-badge

Proposal: add badge for Travis CI example status

Merge pull request #2441 from topecongiro/issue-2438

Return `None` when format_code_block may have failed

Merge pull request #2428 from PSeitz/master

scale WidthHeuristics by max_width

Adding badge and explanation for Travis CI example status

apply some clippy suggestions

Return None when format_code_block may have failed

Merge pull request #2440 from topecongiro/issue-2439

Disable formatting macro-def if args do not fit on one line

Merge pull request #2435 from davidalber/fix-option-typo

Fixing command-line option name

Merge pull request #2187 from brotzeit/master

don't print verbose output when formatting with stdin

Disable formatting macro-def if args do not fit on one line

scale WidthHeuristics by max_width

scale WidthHeuristics by max_width

Preserve trailing two whitespace in comments

Closes #2434.

Fixing command-line option name

Update comment to say that Modified is for internal use only.

Update after review comments.

Move the modified test files to the new location.

Merge remote-tracking branch 'origin/master' into difflines_mode

Merge pull request #2409 from davidalber/configuration-check-opt-out

Skipping formatting of some configuration snippets

Formatting snippets without configuration option marked as skip

don't print verbose output when formatting with stdin

Merge pull request #2432 from davidalber/remove-unused-import

Removing unused import

Merge pull request #2433 from davidalber/unhork-snippet-tests

Fixing path to Configurations.md

Fixing path to Configurations.md

Issue #2419 changed the directory in which
`configuration_snippet_tests` runs, which broke the test. Since the
test is currently annotated with `#[ignore]`, the break wasn't
caught before merge.

Removing unused import

Merge pull request #2431 from davidalber/fix-indent-style-control-flow-snippet

Fixing `indent_style` control flow examples

Merge pull request #2419 from topecongiro/core

Separate rustfmt into multiple crates

Fixing  control flow examples

Fix appveyor

Pub use Config and Summary

Cargo update

Create git-rustfmt crate

Create rustfmt-format-diff crate

Create rustfmt-bin crate

Create cargo-fmt crate

Create rustfmt_config crate

Create rustfmt_core crate

Merge pull request #2417 from topecongiro/issue-2415

Avoid orphan in chain with punctuation

Merge pull request #2416 from topecongiro/eshell

Explicitly disable colored output when it is not supported

Avoid orphan in chain with punctuation

Explicitly disable colored output when it is not supported

Merge pull request #2393 from RReverser/macro_rules

Format stable macro_rules

Add a test for #2208

Closes #2208.

Merge pull request #2412 from topecongiro/issue-2399

Do not reorder items with '#[macro_use]'

Merge pull request #2413 from topecongiro/issue-2401

Use correct offset when unindenting code block

cargo run cargo-fmt

Reformat codebase with current version to pass self_tests (formats macros without repetitions).

Extract branch rewrite function

Format

Move ; between macro branches to a separator

Fix comment handling in macros

Comments WIP

Keep delimiter as part of macro args list

Add repetition example

Omit newline for empty macro branches

Optimise common `=> {{` macro pattern

Revert comments

Support compact macros 2.0 representation

Update some macro tests

Add macro_rules tests

Initial support for macros 1.1