Add two new whitespace options (#1109)

* An option to leave a space before the colon in a type annotation

* An option to leave a space before the colon in a trait or lifetime bound

Merge pull request #1052 from julienXX/fix-imports-with-absolute-paths

Fix imports with absolute paths

Fix imports with absolute paths

Fix overlong function signature (#1089)

* Fix issue-1049

* Add testcase suggested by pepyakin

* Fix last commit

* Handle special case

* Remove debugging println

* Fix grammar in comment

* Change word in comment

* Add test for long func without ret type

* Add one more test

Multiple config file names feature (#1101)

* Add multiple configuration file names feature

* Add '.rustfmt.toml' in README file

* Clean up configuration file code

* Make config file names constant

* Use only one blank line

Fix overlong impl (#1091)

* Fix issue-1048

* Take possible where-clause into account

* Move test to existing test set

* Fix wrong variable name

Add `use` declaration re-ordering (#1104)

* Add config options for combinations of lines and items

* Reordering of import lines implemented.

* Changed nested matches to tuple pattern matching

* Added ordering of path list items to the ordering of use declarations

* Move `format_imports` and `format_import` methods to `imports.rs`

* Add comment to explain how `use` declarations are split off while walking through a module

* Change `ImportReordering` config option to separate boolean options

Canonicalize file paths within the map of file line ranges (#1098)

* Canonicalize file paths within the map of file line ranges

* Forgot to run the tests - and of course, the formatting of the canonicalization change was off, but it's fixed now!

* Move imports to the top of the file, as per @nrc.

* Change `canonicalize_path_string` to return `Option<String>`, `None` indicating an error rather than an empty string

* `format!` is better than string concatenation...

* Change `canonicalize_path_string` to return `Result` rather than `Option`

Merge pull request #1100 from studoot/add-appveyor-support

Add Appveyor CI support

Add appveyor CI support

Merge pull request #1088 from sinkuu/issue_1086

Fix formatting empty block comments (`/**/`)

Fix formatting empty block comments (`/**/`)

issue #1086

Merge pull request #1084 from johannhof/rustfmt-not-found

Show more helpful error if rustfmt is not in PATH.

Show more helpful error if rustfmt is not in PATH.

This fixes #1071.

Merge pull request #1083 from KaivoAnastetiks/fix/easy-issue-repo

Links to the rust-lang-nursery/rustfmt issue list.

rustup

Links to the rust-lang-nursery/rustfmt issue list.

Merge pull request #1075 from johannhof/diff-exit

Return failure exit code on found diffs (fix #906)

Return failure exit code on found diffs (fix #906)

This changes rustfmt to return exit code 4
when run with write mode diff and differences between
the formatted code and the original code are found.

Useful for CI to make sure your contributors actually ran rustfmt.

Merge pull request #1064 from dato/patch-2

Fix typo in path: ~/cargo/bin → ~/.cargo/bin

Fix typo in path: ~/cargo/bin → ~/.cargo/bin

Merge pull request #1054 from sanxiyn/loop-width

Fix width computation in Loop::rewrite

Fix width computation in Loop::rewrite

Include git commit and worktree status in version output (#1060)

This will help in debugging issues as rustfmt gets more users.

If the working tree is clean, output looks like

    $ target/debug/rustfmt -V
    0.5.0 (9f5ed3b)

If the working tree is dirty, output looks like

    $ target/debug/rustfmt -V
    0.5.0 (9f5ed3b worktree dirty)

If git is unavailable, output looks like

    $ target/debug/rustfmt -V
    0.5.0 (git commit unavailable)

To avoid rebuilds on changing tests, the build script will only rerun if
files under src/ are changed. This means the actual git status may show
changed files and this would not show up in the version. This should not
be an issue as files not in src/ should not affect the build output.

Merge pull request #1061 from marcusklaas/fix-subtract2

Fix integer underflow in extra_offset

Fix integer underflow in extra_offset

Merge pull request #1059 from marcusklaas/fix-subtract

Fix integer underflow

Fix integer underflow

Merge pull request #1042 from imjacobclark/1014-refactoring-vector-string-to-join

Refactor string collects to itertools join

Refactoring exsisting filter_maps to maps

Refactor string collects to itertools join

Merge pull request #1033 from marcusklaas/assignment-break

Add test for general assignment breaks

Merge pull request #1032 from marcusklaas/else-if-let-overflow

Fix constraints on pattern formatting of else arms

Add test for general assignment breaks

Fix constraints on pattern formatting of else arms

Merge pull request #1030 from nokaa/master

Update Vim integration instructions

Update Vim integration instructions

Merge pull request #1016 from rust-lang-nursery/try-double-indent

Treat chains with just expr? specially.

Add support for the `default` keyword (#1025)

Adds support for Defaultness on impl methods.
Fixes #945

Merge pull request #1023 from kamalmarhubi/diff-color-fix

print_diff: Don't print color codes if output is not a tty

print_diff: Don't print color codes if output is not a tty

On unix, `term::stdout()` just reads the `TERM` environment variable to
decide what features are available. It does not check if the output file
descriptor is in fact a tty. This resulted in printing escape codes when
redirecting output.

Merge pull request #1007 from kamalmarhubi/basic-line-ranges-v2

Add infrastructure for formatting specific line ranges

README: Explain that --file-lines ranges are 1-based

Explain that FileLines cannot be given in rustfmt.toml

This adds a note to both the `--config-help` output for `file_lines`,
and to the panic message on attempting to deserialize a `FileLines`
struct.

codemap: Add module description

Add copyright notices to added files

rustfmt: Add option to specify line ranges for formatting

This commit adds the `--experimental-file-lines` option to rustfmt. This
allows specifying line ranges to format from the command line.

Refs #434

visitor: Handle specified line ranges in visit_stmt

This commit adds a very rough implementation of handling the specified
line ranges in `config.file_lines_map` for statements. It reformats a
statement if its span is fully contained in the set of lines specified
for the file.

The implementation here is intended as a proof of concept, and
demonstration that the machinery added in the preceding commits is
functional. A final implementation would likely hook in via the
`Rewrite` trait.

Refs #434

Add type to represent collection of lines in files

This commit adds a type to represent lines in files, and adds it to the
`Config` struct. It will be used for restricting formatting to specific
lines.

Refs #434

codemap: Add utilities for looking up line ranges of spans

This commit adds extension methods to `Codemap` to allow looking up line
ranges for spans.

Refs #434

utils: Move codemap related utilities to a dedicated module

This commit adds a `codemap` module, and moves the `CodemapSpanUtils`
added in #857 to it. This is preparation for adding more `Codemap`
specific utilities.

Refs #434

visitor: Add debug log for FmtVisitor::visit_stmt()

Derive Debug for the Input enum

Merge pull request #1017 from marcusklaas/tweak-if-else

Format non-statement if-else expressions on a single line

Merge pull request #1022 from kamalmarhubi/update-syntex-syntax

deps: Update syntex_syntax to 0.33

Merge pull request #1015 from srinivasreddy/readme

Add section - Installing from source

deps: Update syntex_syntax to 0.33

This bump brings syntex_syntax support for `..` in tuple and tuple
struct patterns, which come from RFC 1492:
  https://github.com/rust-lang/rfcs/blob/master/text/1492-dotdot-in-patterns.md

These new patterns are not properly handled in this PR; instead #1021
tracks supporting them.

Refs #1021

Format non-statement if-else expressions on a single line

Treat chains with just expr? specially.

Fixes #1004

Add section - Installing from source

Handle pub(restricted) (#1013)

* Handle pub(restricted)

This commit properly handles pub(restricted) as introduced in RFC 1422
[0]. The syntax support was added in #971, but they were not correctly
formatted.

[0] https://github.com/rust-lang/rfcs/blob/master/text/1422-pub-restricted.md

Fixes #970

* Drop #[inline] attribute on format_visibility

* Make newly non-failing functions return String

The change to `format_visibiilty` means that `format_header` and
`format_unit_struct` can no longer fail. Their return type is updated to
reflect that.

Merge pull request #1008 from rust-lang-nursery/hrtb

Don't ignore universal quantification in function types

Don't ignore universal quantification in function types

Fixes #1006

Merge pull request #1009 from rust-lang-nursery/fn-args-1

Fix off by 2 error in function sigs

Fix off by 2 error in function sigs

Fixes #1000

This is a little conservative in some cases, but better than being wrong in others.

Don't put a newline before `?` when it is the second sub-expression in a chain (#1012)

Fixes #1003

Optionally put short struct variants on one line (#997)

Closes #418

Format inline attributes on out-of-line modules (#996)

Fixes #838

Merge branch 'master' of github.com:rust-lang-nursery/rustfmt

Handle parser errors better

Fall back to basic stdout if we can't unwrap a fancy terminal (#995)

fixes #978

Merge pull request #992 from srinivasreddy/master

change  normalise spelling to American version -

Note in te README about compiler version

Closes #990

change  normalise spelling to American version - normalize as American english is more common across programming community

Write each file as it is formatted (#991)

The old behaviour stored everything in memory until we were finished. Now we write as soon as we can.

This gives better behaviour when formatting large programs, since there is some progress indication. It also opens the door to optimising memory use by not storing everything in memory unless it is required (which it still might be). That is left as future work though.

0.5 release

Merge pull request #893 from marcusklaas/try-shorthand

Try shorthand

Bootstrap it. Hard.

Add try macro to try shorthand conversion tests

Format try shorthand

Merge pull request #981 from DanielJCampbell/generated

Altered FmtVisitor to function correctly on generated code

Altered FmtVisitor to function correctly on generated code

Merge pull request #980 from DanielJCampbell/visibility

Changed modules & visitor mods to public, set filemap::write_file public

Changed modules & visitor mods to public, set filemap::write_file public

Merge pull request #979 from kamalmarhubi/update-syntex-syntax

deps: Update syntex_syntax to 0.32.0

deps: Update syntex_syntax to 0.32.0

Merge pull request #974 from sanxiyn/unused-trait-import

Remove unused trait imports

Remove unused trait imports

Merge pull request #973 from habnabit/fix-associated-items

Don't misplace the :: on associated items.

Don't misplace the :: on associated items.

The rewritten location of the :: on global paths for qpaths was wrong.

Merge pull request #971 from kamalmarhubi/update-syntex-syntax

deps: Update syntex_syntax to 0.31.0

Handle attributes on modules (#968)

* Handle attributes (including doc comments) on inline modules

Closes #22
Closes #684

* Tweak the rules for changing indentation in comments (to do it less often).

Handle new visibility types more gracefully

deps: Update syntex_syntax to 0.31.0

Most of the churn on this bump comes from the `Visibility` enum changing
from

    pub enum Visibility {
        Public,
        Inherited,
    }

to

    pub enum Visibility {
        Public,
        Crate,
        Restricted { path: P<Path>, id: NodeId },
        Inherited,
    }

which require taking `Visibility` by reference in most places. The new
variants are not handled at this point.

Refs #970

Merge pull request #966 from MicahChalmer/skip-children-in-plain-write-mode

Always skip children when using Plain write mode

Always skip children when using Plain write mode

Outputting child module contents in the "Plain" write mode does not make
sense, since there is no way to differentiate code that came from a
child module from that which came from the parent file.

Merge pull request #960 from rust-lang-nursery/big-closures

Block indent large closures and field/method chains

Add visual indent tests for chains

Bail out on recovered errors. (#965)

Closes #915
Closes #930
Closes #931

Fix for stable