Add visual indent tests for chains

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

Merge pull request #912 from rust-lang-nursery/pat-simple-mixed

Change the logic around breaking multiple patterns in match arms

Merge pull request #896 from erikjohnston/fn_arg_one_line

Add fn_arg_one_line option

Reviewer comments - mostly fix rewrite_guard

Change the logic around breaking multiple patterns in match arms

Refactor to use the list code, don't preserve original stacking-ness, base vertical vs mixed formatting on complexity of the patterns.

Closes #386

unify format_string and format_file

Empty structs and struct lits (#920)

* Handle empty struct lits

Closes #835

* Don't crash on empty struct defs.

Not a great fix, but better than crashing.

Don't be so aggressie about line-breaking strings (#911)

We will no longer break in the middle of words, only at whitespace or punctuation.

This means we sometimes over-run, but that seems better than some of the bad splits we see.

Closes #369

rustfmt: Simplify match in project file lookup loop

This commit changes the match in `lookup_project_file` to use pattern
guards.

rustfmt: Make error handling more idiomatic

This commit replaces the `Operation::InvalidInput` variant with
`Result`, and uses the `try!()` macro instead of explicit matching.

rustfmt: Move getopts::Options creation to its own function

Update the world

Merge pull request #910 from kamalmarhubi/cancel-errors

macros: Cancel DiagnosticBuilder when not emitting error

macros: Cancel DiagnosticBuilder when not emitting error

The error handling in libsyntax changed to use a `DiagnosticBuilder`
type in the `Err` variant of `PResult`. This type has `emit()` and
`cancel()` methods. Once created, errors must be emitted or canceled; if
not, the `Drop` impl on `DiagnosticBuilder` will panic.

The first syntex_syntax release to include this change was v0.25.0. The
bump from v0.23.0 to v0.29.1 in #847 did not add any `cancel()` calls,
even though at least one was required. There may be others not caught in
this commit.

Tidy up

Fix long generic indent

Add variant to fn_args_layout

Merge branch 'master' of github.com:nrc/rustfmt into fn_arg_one_line

Merge pull request #909 from srinivasreddy/refactor

Refactoring so that we write some error messages to stderr rather than stdout.

use std::error instead std::out

Merge pull request #908 from matklad/simplify-tests

simplify tests

simplify tests

* change layout so that test group `foo` has source files in
`test/foo/source` and target files in `test/foo/target`.

* use significant comments to specify write mode instead of threading
  Option<WriteMode>

Merge pull request #897 from matklad/refactor-run

Refactor run family of functions

make naming more consistent

Fix. rustfmt write to stderr instead stdout

Fix. rustfmt write to stderr instead stdout

Merge pull request #898 from srinivasreddy/spell

Correct spelling of 'style'

make format_string and fmt_lines private

use format_input function in tests

don't read config twice during tests

Correct spelling of 'style'

refactor: unify run and run_from_stdin

minor: use &mut ref instead of by value argument

This also unifies `write_all_files` and `write_file` functions

Add fn_arg_one_line option

If set, arguments will be kept on one line if they fit. Currently only
applies when `fn_args_layout` is set to `Block`.

This commit also fixes a bug where newlines were inserted inbetween
argument brackets when there were no arguments and `fn_args_layout` was
set to `Block`.

Merge pull request #895 from erikjohnston/fn_arg_block_incorrect_indent

Fix incorrect indent with fn_args_layout: Block

Fix incorrect indent with fn_args_layout: Block

The closing parenthesis for the arg list was written on a new line
without first applying the appropriate indent.

Merge pull request #892 from marcusklaas/closed-ranges

Format closed ranges

Version bump - v0.4

closes #870

Merge pull request #883 from marcusklaas/macro-with-name

Properly format macro's with an extra ident

Do not panic on type ascription or try shorthand

Instead, simply format expressions involving these unstable features
as they were found.

Format closed ranges

Properly format macro's with an extra ident

Merge pull request #889 from kamalmarhubi/next-power-of-two

Use checked_next_power_of_two from std instead of custom method

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

deps: Update syntex_syntax to 0.30.0

Merge pull request #886 from matklad/make-illegal-states-unrepresentable

Introduce verbosity enum

deps: Update syntex_syntax to 0.30.0

This bump includes ability to parse inclusive ranges (`a...b`) and the
question mark operator.

Refs #867, #890

Use checked_next_power_of_two from std instead of custom method

NB The custom method returned 0 on an input of 0, which is arguably
incorrect: 0 is not a power of two; the method in `std` returns 1 in
that case.

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

cargo-fmt: don't return zero on failure

minor: use enum instead of pair of booleans

This is a bit more typing, but statically forbids using both verbose and
quiet

cargo-fmt: don't return zero on failure

Merge pull request #885 from marcusklaas/empty-trait-type-params

Remove trailing whitespace for fn types /w non-unit return

Merge pull request #882 from marcusklaas/empty-makro-issues

Fix issues with empty macros with curly braces

Merge pull request #881 from marcusklaas/nightly-update

Fix tuple destructuring in impl fns

Remove trailing whitespace for fn types /w non-unit return

Fix issues with empty macros with curly braces

Fix tuple destructuring in impl fns

Merge pull request #868 from rust-lang-nursery/tuple-wide

Don't apply the function args heuristic to tuple lits.

Merge pull request #864 from Venti-/fn-density

Add option for vertical function argument list

Merge pull request #874 from kamalmarhubi/rustfmt_skip-clarification

readme: Clarify the ways of specifying rustfmt_skip