cleanup: Use const instead of static

Merge pull request #792 from kamalmarhubi/project-file-lookup-error-handling

bin: Improve error handling in project file lookup

docs: Clarify return type of lookup_project_file

Merge pull request #793 from kamalmarhubi/expect-formatting

tests: Use Result::expect() throughout

tests: Use Result::expect() throughout

`Result::expect()` was added in Rust 1.4. Using it tidies up the code,
and also helps by printing error details, eg, printing syntax error
details if a regex fails to compile. It adds a colon followed by the
`Debug` output from any error, making the periods in messages
unnecessary.

bin: Improve error handling in project file lookup

Previously errors were being silently ignored. Eg, if `rustfmt` did not
have permission to read a `rustfmt.toml` file, the default configuration
was used without informing the user.

Merge pull request #787 from rust-lang-nursery/mod-empty

Put empty modules on one line

Merge pull request #791 from kamalmarhubi/canonicalize-path

bin: Canonicalize path before looking for project file

Merge pull request #790 from kamalmarhubi/handle-rustfmt-toml-dir

bin: Properly handle a directories named rustfmt.toml

bin: Canonicalize path before looking for project file

bin: Properly handle a directories named rustfmt.toml

`lookup_project_file` could erroneously find a *directory* named
`rustmfmt.toml` if there was one in its lookup path, and so ignore any
configuration file it should have found further up. The error handling
resulted in this silently using the default configuration.

Put empty modules on one line

Fixes #463

Merge pull request #785 from DarkDrek/fix-#784

Fix for #784 edge case in comment handling

Fix #784

Merge pull request #783 from jwazny/issue-588

Added where_trailing_comma option.

Added tests.

Limit when we emit a trailing where clause comma.

Added where_trailing_comma option.

The default is 'false', since a lot of the time there's only a single
predicate.

Merge pull request #781 from kyeah/itemmac

Format item macros

Merge pull request #782 from markstory/readme-checkstyle

Update the README concerning write modes.

Add item macro tests

Updates to readme based on feedback.

Update the README concerning write modes.

Add checkstyle and more detail on each of the write modes.

Format item macros

Merge pull request #745 from markstory/checkstyle-output

RFC implementation of checkstyle output

Extract helper functions for testing.

These functions help reduce duplication in the test harness and make it
easier to add tests for other write-modes in the future.

Get checkstyle tests passing.

Fix up the checkstyle test so they pass. There is still an issue with
the content, but I think that is caused by a problem with how diffs are
being calculated presently.

Update checkstyle write mode to take Write arguments.

By accepting Write traits we can write tests using StringBuffer.

Start tests for checkstyle.

They don't yet pass, I've clearly misunderstood the existing tests.

Clean up function names for checkstyle output.

Merge pull request #774 from rust-lang-nursery/issue698

Fix an off-by-one error due to the comma in a struct lit field

Fix an off-by-one error due to the comma in a struct lit field

Also allows splitting the field expression on a new line after the field name.

Fixes #698

Remove reference to 1.4 from README

closes #771

tweak README.md

Use PartialEq/Eq implementation instead of match.

Also cleanup from rebasing onto master.

Extract checkstyle output into a separate module.

Rename functions a bit now that they are specific to checkstyle output.

Update based on pull request feedback.

* Extract duplicated logic.
* Make checkstyle errors into warnings.

Encode XML entities.

Remove unused argument.

Fix formatting errors.

Hack together checkstyle output that compiles.

I'm not sure it does something useful yet though.

Start hacking checkstyle output in.

checkstyle now shows up on the option parser, and the code still
compiles/passes tests. Next up will be outputing the XML to stdout.

Merge pull request #769 from svmnotn/write-mode-config

Allow for setting of write-mode via config file. FIxes #215

Allow for setting of write-mode via config file. FIxes #215

Also from @marcusklaas:
 Refactor code output functions

 Specifically, `write_all_files` no longer returns a HashMap. It would sometimes
 contain items, and sometimes be empty. When "fixed" newlines are required, this
 must now be done with a separate call. The tests use this strategy and should now pass!

Merge pull request #766 from ConnorGBrewster/fix-#489

Added option for tighter punctuation in types. fixes #489

Merge pull request #768 from DarkDrek/amend-to-fix-#447

fixes parsing issue #767

Added test case

Changed TypeDensity::WhiteSpace to TypeDensity::Wide
Changed eq_str and plus_str assignments to use a match

Merge pull request #757 from ConnorGBrewster/fix-#650

Fix for #650

fixes parsing issue #767

If where is on the same line as the impl, { is put on the same line fixes #650

factored if clause into a separate function

Merge pull request #765 from DarkDrek/fix-#447-2

Fix #447 2. try

Removed the failing test part since it will work when #754 is accepted

Handle more possible comment position for if else
Extended the test with the new possiblecomment positions

Keep comments between if and else blocks. Fixes #447

Merge pull request #762 from ConnorGBrewster/fix-#760

Where clause is on same line as fn if fn is empty Fixes #760

Changed is_block_empty to has_body

Added option for tighter punctuation in types. fixes #489

Where clause is on same line as fn if fn is empty fix-#760
only applies if fn_empty_single_line is set to true

Merge pull request #756 from cassiersg/missed-comments

Detect (and recover) when comments disappear

Merge pull request #740 from svmnotn/less_tmp_files

stop creating bk files if there are no changes. Fixes #733

stop creating bk files if there are no changes. Fixes #733

Merge pull request #758 from cassiersg/issue-717

Fix issue with windows line endings (#717).

Document comment::*CodeCharKind

Fix issue with windows line endings (#717).

The '\r' character wasn't counted in the line length.

Detect when comments disappear

When the reformatted code doesn't contain the same quantity of comments
as the original code, use the original code instead of the reformatted
code.
This is done for all expressions and `let` statements.

This should be used at the finest grained level possible, to avoid that
a small disappearing comment prevents a big chunk of code to be
reformatted.

Kind of fixes (avoid disappearing comments, but prevents a good
formatting is such case) #285 #225 #563 #743

Merge pull request #749 from sanxiyn/skip-arm

Add commas after skipped arms when needed

Add commas after skipped arms when needed

Merge pull request #747 from Manishearth/clippy

Clippy rustfmt

Clippy rustfmt

Merge pull request #744 from sanxiyn/wildcard-arm

Trailing commas for wildcard arms

Trailing commas for wildcard arms

Merge pull request #739 from svmnotn/case_insensitive_config_options

ignore case for config enums. Fixes #738

ignore case for config enums. Fixes #738

Merge pull request #736 from sinhpham/vsc_doc

Add doc for Visual Studio Code plugins

Merge pull request #734 from sinhpham/issue_700

Fixed #700

Add doc for Visual Studio Code plugins

Fixed #700, use default color for titles and context in write-mode=diff

Merge pull request #732 from utkarshkukreti/update-diff-to-0.1.8

Update diff to 0.1.8

Add tests for #606.

Bump diff to 0.1.8. Fixes #606.

Merge pull request #729 from marcusklaas/funky-self

Make fn argument formatting more resilient for complex self types

Make fn argument formatting more resilient for complex self types

Merge pull request #728 from marcusklaas/typed-mut-self

format typed mut self

Merge pull request #726 from marcusklaas/string-backslashes

Prevent incorrect backslash removal in strings

Reduce dependency on pprust

Preserve mutability for typed self arguments

Prevent incorrect backslash removal in strings

Merge pull request #722 from matklad/test-ignore-children

test ignored out of line modules

test ignored out of line modules

tests #719

Merge pull request #720 from matklad/update-lcok

update Cargo.lock to match Cargo.toml

Merge pull request #719 from matklad/ignore-child-mods

add option to ignore out of line modules

update Cargo.lock to match Cargo.toml

add option to ignore out of line modules

Merge pull request #713 from JanLikar/master

Add verbose and quiet flags to cargo-fmt

Add verbose and quiet flags to cargo-fmt

Quiet mode supresses rustfmt's stdout, verbose mode prints targets that
are going to be formatted.

Merge pull request #710 from JanLikar/master

Remove `--write-mode=replace` from cargo-fmt

Remove `--write-mode=replace` from cargo-fmt

Since replace is the default rustfmt write mode, there's no need to
call rustfmt with `--write-mode=replace`. As a bonus, it is now also
possible to override the write-mode.

Merge pull request #704 from Sean1708/patch-1

Fix spelling mistake in cargo-fmt usage string.

Fix spelling mistake in cargo-fmt usage string.