Fix bug in identifying comments

Fixes 1233

Add a more lightweight method for rewriting comments when we are not normalising

Fixes #652

Allow empty blocks on one line in more places

In particular if they contain only a single-line comment.

Fixes #493

make rustfmt-normalize_comments false by default

Refactor if/else formatting

Removes else_if_brace_style from config options. Use control_brace_style instead.

Refactor pairs/binops

Fix #1258 (#1266)

* Fix #1258

* Add test

Do not ignore space_before_bound in where clause (#1267)

Better message for LineOverflow error (#1268)

(with current length and configured limit)

Remove `self` from `use foo::bar::self;`

Also adds the `normalize_imports` config option.

Fixes #1252

If a string goes over-width, give up formatting it

Logic here is that the original might be better and at best we'll only make it a different kind of bad.

Fixes #1237

Macros in types are possible

Fixes #1234

Fix #1259 (#1263)

* Use `Indent::block_indent` instead of adding `config.tab_spaces`

* Fix #1259 impl block closing brace placement

Merge pull request #1251 from erickt/master

Version bump for dependencies, add support for `break` values

Version bump for toml, unicode-segmentation, itertools, and walkdir

Update to the latest syntex module

One notable feature is this this adds support for the experimental
`let x = loop { ... break $expr; }` syntax. This also includes a
test for formatting all the break and continue variations.

Fixed broken link to style guidelines (#1245)

Recognize "rlib" and "cdylib" crates (#1238)

Addresses #828 and https://github.com/lunaryorn/emacs-module.rs/issues/4 and probably numerous others.

Don't drop opening brace on long line matches. (#1228)

* Don't drop opening brace on long line matches.

Fixes #1225

* Added a test case for long match arms with braces on newline.

Indent open brace for impl when nested. (#1227)

* Indent open brace for impl when nested.

Fixes #1226

* Added test case for indented impl with brace on newline

Merge pull request #1231 from mineo/condense_wildcard_suffices

Prevent an ugly linebreak in --config-helps description of condense_wildcard_suffices

Prevent an ugly linebreak in --config-helps description of condense_wildcard_suffices

update Serde, etc.

Lots of changes to how closures work

v0.6.3

Stripping trailing commas and spaces from `vec!` elements (#1219)

* Stripping trailing commas and spaces from `vec!` elements

* Stripping trailing commas and spaces ONLY from `vec!` elements

* Added comment

Merge pull request #1221 from Zarokka/issue-1216

fix #1216: panic on comment starting with //*

Merge pull request #1218 from dereckson/patch-1

Improve config::WriteMode comments

fix #1216: panic on comment starting with //*

Improve config::WriteMode comments

- Fix typo
- Switch comments verbs to the the 3rd person to offer a better coherence among options

Merge pull request #1212 from marcusklaas/test123

Condense suffix strings of simple wildcards in patterns

Condense suffix strings of simple wildcards in patterns

Merge pull request #1207 from Fraser999/master

Fixes #1184 where reordering import items could cause a panic.

Merge pull request #1206 from masonium/fmt-extra-files

#1126 cargo-fmt processes bench, test, example files

Fixes a few doc typos.

Fixes #1184 where reordering import items could cause a panic.

Also extends an existing test to avoid regression.

cargo-fmt processes bench, test, example files

Merge pull request #1200 from efyang/strayspace

Remove extra whitespace after macro calls

Remove extra whitespace after macro calls

Stop extra newlines from being added after block comments (#1185)

Add spaces_within_square_brackets config option. (#1191)

* Add spaces_within_square_brackets config option.

Enabling the config enforces spaces within various array/slice brackets.

* Fixed budget-calculations for [] spacing

Merge pull request #1188 from Rantanen/master

Add support for spaces_within_angle_brackets

Merge pull request #1189 from Pursuit92/master

Quick fix for "impl Trait"

Quick fix for "impl Trait"

* Remove "TODO" conversion
* Add a couple of simple test cases

Fix courtesy of @signaraiona

Added test for `for< 'a >`

Add support for spaces_within_angle_brackets

Enabling the config adds spaces within type arguments: <>

Add support for spaces_within_parens config (#1187)

* Add support for spaces_within_parens config

* Changes based on review comments

Merge pull request #1182 from philipc/issue-1149

Preserve macro formatting if we can't rewrite it

Merge pull request #1186 from woshilapin/vim-doc

Update link towards vim documentation

Update link towards vim documentation

cargo update

Merge pull request #1183 from oylenshpeegul/patch-1

Link to rust-mode.el instead of rustfmt.el

Link to rust-mode.el instead of rustfmt.el

Link directly to rust-mode.el, since rustfmt.el is marked obsolete.

Preserve macro formatting if we can't rewrite it

Add custom comments (#1179)

* Add custom comments

This allows users to use custom comments such as

```
//@ this is a custom comment
//@ with multiple lines
```

without having them destroyed by rustfmt.

* Fix issues with empty lines

* Check non-whitespace right after custom comments

Merge pull request #1175 from nokaa/master

Fix broken import

Fix broken import

Write to the supplied buffer if one is supplied to format_input

Merge pull request #1166 from sinkuu/fix_775

Fix #775: Off-by-one overrun in deeply nested function call

Fix #775

rewrite_tuple referring config.max_width where given width should be
used

Merge pull request #1156 from sinkuu/break_if_let

Prefer line break at `=`/`in`

Merge pull request #1165 from sinkuu/fix_self_tests

Fix self test

Fix self test

Merge pull request #1162 from sinkuu/fix1040

Fix #1040

Merge pull request #1161 from sinkuu/fix550

Fix #550: `if` nested in tuple is indented oddly

Merge pull request #1160 from est31/master

Two improvements

Prevent line breaking at `=` or `in` after trivial patterns

Fix #550: `if` nested in tuple is indented oddly

Improve comment rewriting with normalize_comments == false

Only change multiline comments of the form

```rust
/*
 * Text
 */
```

while not affecting comments of the form

```rust
/*
Text
*/
```

when normalize_comments is off. In the first case,
we have a known character we can align against, while
we don't have one in the second case.

Before, we have converted the second form into the first,
but this is against the spirit of normalize_comments being
turned off.

Fixes #956

Add three new options for spaces

Update to latest Syntex

+ Cargo update
+ 0.6.2 release

Merge pull request #1151 from sinkuu/issue1021

Fix #1021: Handle `..` in tuple / tuple struct patterns

Fix #1040

Fix #1021: Handle `..` in tuple / tuple struct patterns

Split impl at 'for' if a line break is needed (#1148)

* Split impl at 'for' if a line break is needed

* Fix formatting

* Improve comments

* Skip second try if there is no 'for'

* Restore intentional trailing whitespace

* Change test source to be incorrectly formatted

* Restore more missing trailing whitespace

* Remove too much whitespace...

Really should learn how to use git revert.

Merge pull request #1147 from brettcannon/patch-1

Put rustfmt on PATH so `cargo fmt` works on Travis

Put rustfmt on PATH so `cargo fmt` works on Travis

Closes #1143

Merge pull request #1135 from sinkuu/clippy

Run clippy

Run clippy

Merge pull request #1134 from sinkuu/issue977

Fix #977

Fix #977

Merge pull request #1142 from kamalmarhubi/fix-1140

Check term supports colour before printing fancy diffs

Check term supports colour before printing fancy diffs

For a terminal like DUMB, we were still attempting to print colourful
diffs, which failed with a `NotSupported` error.

Fixes #1140

Merge pull request #1138 from johannhof/travis-docs

Add instructions on how to run rustfmt on Travis

Add instructions on how to run rustfmt on Travis

This adds very simplistic instructions on how to run rustfmt on CI.

I also wrote a blog post for more details: http://johannh.me/blog/rustfmt-ci.html

Fix issue 1124 - detect start of output rather than start of input file when writing output source file (#1133)

* Change required to prevent a trailing space at the end of a separate module being propagated

* Detect the start of the output file rather than the start of the input file when deciding whether to output preceding snippets - this stops unnecessary whitespace and blank lines from being inserted when spans and statements are output in an order other than that from the input file.

* Add code to prevent space from being added with the prefix snippet if a) the snippet is entirely horizontal whitespace, or b) the snippet contains whitespace  followed by a newline. This prevents trailing spaces at the end of a line from being added.

* Tests for this issue

* Tidy up `match` statements

* Add test with blank lines between `use` statements

v0.6.0

Merge pull request #1123 from juicejitsu/807

Don't emit filename in diff mode, add filename to diff metadata

Fix #1120 - rework how the use list prefix is determined. (#1121)

* Fix #1120 - rework how the use list prefix is determined.

* Added test commentary & another test case

Merge pull request #1122 from studoot/add-test-for-1111

Add test case for issue #1111

Don't emit filename in diff mode, add filename to diff metadata

Add commentary

Reformat the source to actually pass the tests!

Add test case for issue #1111, by adding another route by which a test file's config can be located

Merge pull request #1119 from HighCommander4/issue1109b

Apply space_before_type_annotation to struct ctors (#1109)

Apply space_before_type_annotation to struct ctors (#1109)

Merge pull request #1117 from pepyakin/issue-1116

Fix underflow in format_imports

Fix issue-1116

Merge pull request #1110 from HighCommander4/issue1109a

Add two new whitespace options (#1109)

Merge pull request #1118 from julienXX/change-multirust-to-rustup-in-readme

Update README with Rustup.rs instead of multirust

Update README with Rustup.rs instead of multirust