Implement combining openings and closings

Merge pull request #1565 from topecongiro/issue-1563

Implement vec![expr; expr] and avoid unnecessary indent in chain

Update tests

Format source codes

Refactor source codes

Allow macro rewrite to fail on rhs

Fix weird indentaion in chain

Implement 'vec![expr; expr]'

Merge pull request #1558 from mjkillough/config

Add a flag to emit which options are used

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

Merge pull request #1569 from mathstuf/example-typos

Configurations: fix typos in example signatures

Merge pull request #1568 from mathstuf/suffixes-typo

config: fix `suffices` -> `suffixes` typo

Configurations: fix typos in example signatures

config: fix `suffices` -> `suffixes` typo

Fixes #1477.

Merge pull request #1562 from flier/reorder_imports_in_group

reorder imports in group

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

Add --dump-default-config and --dump-minimal-config.

 - `--dump-default-config` outputs the default configuration to the
   specified file as TOML and then exits.
 - `--dump-minimal-config` is checked after formatting files as normal.
   If present, any configuration options that were checked during
   formatting are written to the specified file as TOML.
 - These options were added only to `rustfmt`, not to `cargo fmt`. They
   can be specified when using `cargo fmt` by placing them after `--`.
 - It would have been nice if the filename was optional, so you could
   run just `rusfmt --dump-minimal-config build.rs` to have it output to
   `rustfmt.toml`. However, this doesn't do what you might expect: it
   outputs the config to `build.rs`!

reorder imports in group

Return `PartialConfig` from `Config` methods.

Leave serialization to the caller, but provide a
`PartialConfig.to_toml()` method, to deal with the fact that
`file_lines` can't be serialized.

Add a simple test.

Provide `config.set().item(value)` API.

This API isn't fantastic, but it's the best I can come up with without
something like `concat_idents!()`. There are relatively few places where
config is set, to hopefully the ugliness isn't disastrous.

Change previous occurences of `config.item = value` to this new API,
rather than using `config.override_value()`. Undo the changes to
`override_value()`, as it's no longer important to propogate the error
to the caller. Add a test for the new interface.

Use unreachable!() instead of panic!().

Remove ConfigTracker - inline in Config.

Use an individual `Cell<bool>` on each config item, rather than
maintaining a set of config names that were accessed. Add a simple unit
test.

Merge pull request #1560 from topecongiro/issue-1441

Use multiline when signature does not fit single line

Bump version of strings.rs and cargo update

Use multiline when signature does not fit single line

Add note about libsyntax branch to README

Merge pull request #1559 from topecongiro/poor/chain

Allow the last element in chain to go multiline

Merge pull request #1557 from mjkillough/issue_1366

Add test for issue #1366.

Format source codes

Allow last child in chain to go multiline

Add methods to serialize Config to TOML.

Two different modes:

 - Serialize the full Config object. This is useful as
   `Config::default().to_toml()` to output a rustfmt.toml with defaults
   (#317).
 - Serialize only the options that have been accessed. This could be
   useful to output a minimal rustfmt.toml for a project. (If the
   default value of any unused config item changes, you'll then get the
   new default when you come to use it).

This commit doesn't expose this anywhere - deciding a sensible CLI is a
bit trickier.

This commit also has very simple error reporting (Result<String,
String>) - once the CLI is decided, a more sensible method of reporting
errors might become obvious.

Track which config items are accessed.

Required by #865. This doesn't introduce any method to view which
parameters are accessed.

We record which config items are accessed even if we don't intend to
output them, as we assume it will be a relatively cheap operation.

Add test for issue #1366.

As the comments in the issue state: it appears to work now, so this test
should just stop any regressions.

The second example in the issue description will format to the first,
which is then a fixed-point.

Switch to accessing config items via method.

Preparation for #865, which proposes adding a flag which outputs which
config options are used during formatting.

This PR should not make any difference to functionality. A lot of this
was search-and-replace.

Some areas worthy of review/discussion:

 - The method for each config item returns a clone of the underlying
   value. We can't simply return an immutable reference, as lots of
   places in the code expect to be able to pass the returned value as
   `bool` (not `&bool). It would be nice if the `bool` items could
   return a copy, but the more complex types a borrowed reference... but
   unfortunately, I couldn't get the macro to do this.
 - A few places (mostly tests and `src/bin/rustfmt.rs`) were overriding
   config items by modifying the fields of the `Config` struct directly.
   They now use the existing `override_value()` method, which has been
   modified to return a `Result` for use by `src/bin/rustfmt.rs`. This
   benefits of this are that the complex `file_lines` and `write_mode`
   strings are now parsed in one place (`Config.override_value`) instead
   of multiple. The disadvantages are that it moves the compile-time
   checks for config names to become run-time checks.

Merge pull request #1556 from topecongiro/issue-1555

Only add offset for unary op

Format source codes

Use offset_left for rewrite_unary_prefix

Merge pull request #1510 from fintelia/better-filelines

Better file_lines

Merge pull request #1551 from topecongiro/else-if-early-line-break

Forbid early line breaks inside else if

Merge pull request #1550 from topecongiro/issue-1547

Prevent rewriting closure block to expr inside macro

Merge pull request #1552 from guanqun/give-specific-errors

add more specific reason why it fails to parse

add more specific reason why it fails to parse

Add and improve tests of file_lines

Refactor file range checking

Format source code

Use precise width when rewriting else if

Prevent rewriting closure block to expr inside macro

Merge pull request #1546 from topecongiro/bug/range-full

Add a missing trailing comma before elipses

Add a missing trailing comma before elipses

Merge pull request #1530 from azyobuzin/vscodeextlink

Change the link of Visual Studio Code extension

Merge pull request #1543 from topecongiro/field-attr

Support struct_field_attributes

Add a link of rls_vscode

Support struct_field_attributes

Merge pull request #1540 from topecongiro/fn_call_width

Use vertical layout when args len is larger than fn_call_width

Use vertical layout when args len is larger than fn_call_width

Merge pull request #1529 from topecongiro/issue-1528

Use newline only when fn_call_style is Block

Try multi-line only when fn_call_style is Block and rewriting function call

Change the link of Visual Studio Code extension

RustyCode is no longer maintained.

bump version (0.8.4) and cargo update

Merge pull request #1523 from topecongiro/poor-formatting/fn_call_style

Use multiline when args exceeds max_width in fn_call_style = "block"

Merge pull request #1522 from topecongiro/poor-formatting/if-else

Inherit alignment to 'else'

Use multiline when args exceeds max_width in fn_call_style = "block"

Inherit alignment to last else

Refactor rewrite_pat_expr

Merge pull request #1519 from topecongiro/poor-formatting/if-else

Remove visual_indent when rewriting else block

Merge pull request #1520 from topecongiro/allow-match

Apply closure exception to match in args

Apply closure exception to match in args

Remove visual_indent when rewriting else block

Merge pull request #1516 from topecongiro/remove-try

Replace 'try!' with '?'

Replace 'try!' with '?'

Merge pull request #1506 from topecongiro/issue-1351

Use block indent if visual indent exceeds max_width when rewriting block

Use block indent when visual indent failed

Merge pull request #1442 from topecongiro/poor-formatting/trailing-comma

Remove comma from function definition with a single argument

Merge pull request #1511 from topecongiro/bug/closure-fallback

Add fallback path inside rewrite_closure

Merge pull request #1513 from topecongiro/refactoring

Add methods to Shape

Refactor Shape

Add indented and with_max_width

Merge pull request #1512 from topecongiro/config-path

Allow config-path to point to a file

Allow config-path to point to a file

Add fallback path

Respect file_lines within write_snippet() function

Respect file_lines within format_lines() function

Block format control flow discriminant expressions and binops

Fixes #1450

Adds control_style option

Update tests

Merge pull request #1502 from topecongiro/tests

Add tests for issues which can be closed on master

Use SeperatorTactic::Never for a single argument

Add tests for issues which can be closed on master

Merge pull request #1495 from topecongiro/poor-formatting/closure

Fix weird indentation inside closures

Merge pull request #1500 from shioju/master

Format macros in trait item position

Merge pull request #1496 from est31/master

Drop rustc-serialize, use serde, update toml dependency

Format macros in trait item position

Check format failures explicitly in visit_block

Format fix

Simplifications

Drop rustc-serialize, use serde, update toml dependency

cargo update

Update tests

Format source codes

Keep a chain with length chain_one_line_max in a single line

Split a long chain with a single child