Merge pull request #189 from nrc/match

match expressions

Re-jig binop formatting

and misc other fixes from the reviews.

reformatting/rebasing

Handle span error with `continue`

This should be properly addressed by #184, but requires a change to the rustc parser, so this patch just works around the issue.

Rewrite match expressions (continued).

Formatting

Allow `{}` to remain.

Rewrite match expressions

Merge pull request #178 from marcusklaas/single-config-override

Allow override of single configuration lines in tests

Add option to override single configuration lines for tests

Merge pull request #176 from marcusklaas/no-backup

Add an option to forgo backups

Add an option to forgo backups

Merge pull request #188 from marcusklaas/refactor-itemize-list

Refactor itemize list so that it produces an iterator

Refactor itemize list so that it produces an iterator

Merge pull request #193 from posix4e/doc-defaulttoml

Doc defaulttoml

Provide information about default.toml

Merge pull request #174 from marcusklaas/format-paths

Format paths

Format paths

Merge pull request #159 from marcusklaas/explicit-self

Correctly format typed self in function arguments

Correctly format typed self in function arguments

Merge pull request #162 from cassiersg/sep-mods

Move modules listing outside of FmtVisitor

Fix nits for filemap

Kill ChangeSet and replace remaining part by FileMap

Also fix style issues
Cargo update to get fixes of strings.rs (was cause of a misformatted
function call).

Remove ChangeSet of FmtVisitor

Separate modules and files listing of reformatting

As suggested in #141
Closes #156

Merge pull request #167 from marcusklaas/unsafe

Properly format unsafe blocks

Properly format unsafe blocks

Merge pull request #164 from marcusklaas/rustup

Rustup

Phase out token::get_ident in favour of ToString

token::get_ident was removed in rust commit 00a5e66f818ad9d79cc4425f5564c7b07e3213a6

Merge pull request #144 from marcusklaas/single-use

Format simple imports

Merge pull request #161 from cassiersg/fix-bin-op-width

Add regression tests for binOps

Add regression tests for binOps

Format simple imports

Merge pull request #142 from cassiersg/rewrite-imports

Move 'use' to Rewrite

Add a helper method to format imports

Indent fix

Move 'use' to Rewrite

Implements Rewrite for ViewPath

Behavior change: always use max_width instead of ideal_width for use
list rewrite. I think it looks better, was also suggested by @nrc in
https://github.com/nrc/rustfmt/issues/82#issuecomment-105314265

Fix width computation in rewrite_binary_op

Operator width was counted twice.

Merge pull request #147 from marcusklaas/even-more-expr

Format if expressions & loops

Merge pull request #158 from marcusklaas/modules

Use new module code from libsyntax

Use new module code from libsyntax

Refactor some things; add extra tests.

Merge pull request #157 from marcusklaas/box-syntax

Preserve box syntax

Preserve box syntax

Merge pull request #151 from marcusklaas/fix-for-servo

Fix underflow bug in rewrite_call

Fix underflow bug in rewrite_call

Fixes https://github.com/nrc/rustfmt/issues/148.
Now properly propagates the rewrite failure instead of panicking. Added regression test from servo code. This example will be properly rewritten when https://github.com/nrc/rustfmt/issues/17 is addressed.

Simplify CompilerCalls impl

Merge pull request #146 from marcusklaas/fix-inverted-span

Format unnamed function arguments

Format unnamed function arguments

Format all the loops!

Format if-let-else expressions

Format if-else expressions

Format while loops, including labels

Resolve some warnings

Merge pull request #137 from nrc/struct-lit-opt

Options for formatting struct literals

Merge pull request #139 from nrc/todo-sorry

Don't apologise if its not our fault in warnings

Tidy up some overrunning lines

Don't apologise if its not our fault in warnings

Fix some bugs

Reformat code to new struct lit style

Add an option to format struct lits with either block or visual indent

Merge pull request #128 from marcusklaas/subexpr

Format loops

Format loops

Merge pull request #132 from cassiersg/fix-tup-lit

Fix bug in rewrite_tup_lit

Fix bug in rewrite_tup_lit

Wasn't using all width available when the lenght is more than 1.

Merge pull request #122 from cassiersg/fix-mod

Reformat modules in external files

Run rustfmt on the code

Format modules into separate files

Add comma after struct-like enum variant

This is mandatory.

Fix fn decl rewriting in case of generics

An opening paren in generics caused a false-positive detection of args
beginning. The result was the creation of comments with some code into
it.

Merge pull request #124 from cassiersg/fix-bin-op

Fix bugs in width computation for bin op

Fix bugs in width computation

Merge pull request #99 from marcusklaas/bool-expr

Format expressions with binary and unary operators

Format expressions with binary and unary operators

Merge pull request #113 from marcusklaas/import-comments

Update import list formatting

Update import list formatting

Include comments and sort items alphabetically.

Merge pull request #93 from marcusklaas/tuple-structs

Format tuple like structs

Fixup comment wrapping in lists

Format comments in struct literals

Format tuple-like structs

Merge pull request #112 from marcusklaas/config-fix

Remove global mutable config to allow for concurrency

Remove global mutable config to allow for concurrency

Merge pull request #90 from marcusklaas/empty-imports

Remove empty list imports

Merge pull request #107 from cassiersg/rewrite

Add a Rewrite trait

Use FromIterator implementation for Option

Combined with try_opt!, this avoid an explicit for loop or another macro.

Comments for Rewrite

Add a try_opt! macro for ease of work with Rewrite

A basic impl of Rewrite for ast::Expr

Merge pull request #114 from marcusklaas/rustup

Rustup

Merge pull request #117 from cassiersg/fix-round-power-of-two

Use wrapping operations for bit-level functions

Use wrapping operations for bit-level functions

This handles a underflow panic on round_up_to_power_of_two(0).

Rustup

Merge pull request #109 from marcusklaas/fix-string-lit

Fix off-by-one errors in rewrite_string_lit

Merge pull request #96 from marcusklaas/find-uncommented

Deal with problematic characters in comments

Add tests for string literal formatting

Fix off-by-one bugs in `rewrite_string_lit`

Multi-line literals would typically have a character too many. Splitting of escape sequences also wasn't working correctly.

Deal with problematic characters in comments

Merge pull request #106 from marcusklaas/issue-number-check-2

Implement checks for unnumbered TODOs and FIXMEs

Implement checks for unnumbered TODOs and FIXMEs

Merge pull request #101 from nrc/test

Add a terrible, but useful, test which I used locally