return non-zero exit code if there were errors

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

readme: Clarify the ways of specifying rustfmt_skip

It was previously unclear that the `cfg_attr` version works in stable
Rust.

Merge pull request #866 from rust-lang-nursery/empty-extern

Keep empty extern blocks on one line

Merge pull request #869 from ConnorGBrewster/add-trait-formatting

Add trait formatting

Fixed nitpicks
Removed unneeded comment
Commented line wrapping conditions
Removed prefix param on `rewrite_associated_type`
Merged `rewrite_associated_static` into `rewrite_static`
Added extra tests

Fixed span_after issue

Merge branch 'master' of https://github.com/rust-lang-nursery/rustfmt into add-trait-formatting

Readded spaces to source test

Added tests

Finished implementing impl and trait type/const

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

Closes #450
Closes #510

Keep empty extern blocks on one line

Closes #462

cargo update

Merge pull request #857 from kamalmarhubi/codemap-ext

utils: Add CodeMapExt extension trait for span_* methods

Add independent test for Vertical rustfmt-fn_args_density

Fix last comment turning into block comment

This fixes the comment after the last argument in an argument list
being turned into a block comment when it's not necessary.

Add test for vertical fn_args_density

Add Vertical fn_args_density

This adds new option Vertical to fn_args_density, which formats the
list vertically no matter what.

utils: Add CodeMapSpanUtils trait for span_* methods

This commit adds a CodeMapSpanUtils extension trait on CodeMap, and
moves some functions to methods there:
  - span_after
  - span_after_last
  - span_before

This better reflects them being lookup methods on the codemap.

Merge pull request #859 from solidsnack/master

Write non-output to stderr when there is output

Updated to latest rustfmt

Uncommented missed spans check

Removed unneeded pattern matching

Fixed formatting

Added support for associated type rewriting

Added where clause formatting

Adding trait formatting
trait header formatting finished
generic formatting finished
added basic trait bounds formatting

Merge pull request #851 from Amanieu/const_unsafe

Fix ordering of "const unsafe fn"

Use braces, not parens, for macro def

Align arguments

Write non-output to stderr when there is output

Add a test for #850

Fix ordering of "const unsafe fn"