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

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

Update docs (and a function name)

Add a bootstrap script for running rustfmt on itself.

Change defaults and update tests and source

New defaults are `Tabbed` for `chain_indent` and `chain_base_indent`, and `5` for `closure_block_indent_threshold`.

Tests

Tabbed/Inherit indent for chains works even without a newline after first item.

refactor and document the chain reformatting code

block indent large closures

somewhat rough around the edges

Format visibility of associated consts (#953)

Fixes #951.

Merge pull request #950 from marcusklaas/extern-abi

Add option to force explicit extern ABI's

Add option to force explicit extern ABI's

Merge pull request #948 from flutterhash/master

Added option to configure if/else brace style

Added loop/match brace style control option

This adds Allman style loop/match braces as an alternative to the current
default style.

Added option to configure if/else brace style

Merge pull request #947 from marcusklaas/match-pattern-limit

Fix off-by-one error in pattern formatting

Fix off-by-one error in pattern formatting

rustfmt: Parse options once instead of once per file argument (#944)

`update_config()` was parsing the `write-mode` option once for each file
argument. This commit parses them once up front into a `CliOptions`
struct, which is then applied to the config before calling `run()`.

Merge pull request #941 from marcusklaas/type-ascription

Format type ascription

Merge pull request #942 from marcusklaas/break-continue-fix

Check length of break and continue expressions

Merge pull request #938 from kamalmarhubi/rename-doc-hint

config: Rename get_variant_names to doc_hint

Check length of break and continue expressions

Format type ascription

Don't print empty lines to stderr (#940)

Fix closures again (#937)

* Fix closures again

Closes #934

Oh god, the rules for parsing closures are even more messed up than I thought - whether or not there is an inner block or depends not only on if there are braces, but also if there is a return type for the closure (!) and if there are statements in the block.

* Fix overflow

config: Rename get_variant_names to doc_hint

The `ConfigType` trait is implemented for non-enum types, so the name no
longer makes perfect sense.

Merge pull request #923 from matklad/proper-exit-code

return non-zero exit code if there are errors

return non-zero exit code if there were errors

Merge pull request #936 from marcusklaas/licenses

Add license stuff

Merge pull request #935 from marcusklaas/issue-770

Add regression test for issue 770

Add license stuff

Fix brace indentation for impl items (#927)

Add regression test for issue 770

Merge pull request #928 from matklad/patch-1

don't silence error unnecessary

don't silence error unnecessary

This `if` was used to separate error output from the formatting output, when they both used stdout. It was useful for integration with tools, which can submit input to stdin and read pretty printed result from stdout without worrying about errors intermingled with the actual result.

But now we write errors to `stderr`, so the problem disappears and we can safely remove this `if`.

Errors should never pass silently, unless explicitly silenced.

Merge pull request #926 from rust-lang-nursery/closures

Be careful about where we change braces in closures

Be careful about where we change braces in closures

And some general refactoring of closure code.

Fixes #863

Merge pull request #914 from kamalmarhubi/invalid-operation-result

rustfmt: Make error handling more idiomatic

Change stdout to stderr (#925)

Merge pull request #921 from matklad/more-unification

unify format_string and format_file

Fix the build

Merge pull request #918 from rust-lang-nursery/ty-variadic

Handle variadic function types

Handle variadic function types

Closes #842