Merge pull request #1609 from topecongiro/nested-tuple

Put a space between nested tuple field

Put a space between nested tuple field

Merge pull request #1608 from topecongiro/issue-1598

Fix index bug in write_snippet_inner

Merge pull request #1607 from topecongiro/many-slashes

Allow longer custom comments

Fix index bug in write_snippet_inner

Allow longer custom comment

Merge pull request #1604 from topecongiro/comment-group

Format comments with different opening in different manner

Use different style when rewriting comment with different opener

Merge pull request #1602 from topecongiro/attributes-on-expressions

Allow attributes on expressions

Merge pull request #1601 from topecongiro/issue-1600

Allow attributes in generics of impl

Merge pull request #1599 from topecongiro/performance/binary-search

Avoid unnecessary binary search in rewrite_call

Add tests for issues which can be closed on the current master

This PR adds tests for #325, #1092,  #1214, #1278, #1329 and #1427.

Allow attributes on expressions

Allow attributes in generics of impl

Avoid unnecessary binary search in rewrite_call

Merge pull request #1597 from topecongiro/comments-after-attributes

Allow comments after attributes of struct fields

Merge pull request #1596 from topecongiro/issue-1595

Add a trailing comma if the last arg ends with comment

Merge pull request #1593 from topecongiro/issue-1439

Allow comments after where clause

Merge pull request #1588 from topecongiro/nesting-macro

Allow macro to be nested like function call

Merge pull request #1591 from topecongiro/associated-type-defaults

Allow default associated types

Allow comments after attributes of struct fields

Add a trailing comma if the last arg ends with comment

Allow comments after where clause

Allow default associated types

Format source codes and add tests

Allow macro to nested and overflowed like function call

Set inside_macro to false when rewriting args of macro

Merge pull request #1589 from topecongiro/issue-1587

Put closing paren on the next line when the last arg contains comment

Put closing paren on the next line when the last arg contains comment

Merge pull request #1586 from topecongiro/minor

Fix minor bugs

Merge pull request #1584 from topecongiro/poor/chain-trailing-try

Enhance chain

Add offset when rewriting index

Update tests

Be precise with one_line_budget for fn_base

Add a test for #1380

Format source codes

Add chain_split_single_child option

Allow the last elem in chain to overflow

Refactoring: introduce one_line_budget

Format source codes

Disallow single line chain when the line exceeds the given budget

Consider trailing try operations when counting almost_total

Simplify --version info

Remove dependency on Itertools

Remove multimap dep

Merge pull request #1582 from topecongiro/poor/fn_call_style_block

Decide whether args fit in a single line inside rewrite_call_args

Decide whether args fit in a single line inside rewrite_call_args

Merge pull request #1578 from topecongiro/poor/macro

Use original code snippet for macro which contains invalid syntax

Return original snippet when macro contains invalid syntax

Merge pull request #1576 from topecongiro/chain/float-first-child

 Allow first child to stay on the same line with block parent when rewriting chain

Format source codes

Add heuristic when rewriting chain with a single child

rustfmt splits chain into multiline when the length of chain exceeds
`chain_one_line_max`. However, currenly this rule only applies when the chain
has more than one child. This can lead to unexpected long chain if the parent
is long. This commit adds heuristic that if the length of parent is longer
than the half of `chain_one_line_max` we use multiline even if there is
only a single child.

Format source codes

Use correct width when rewriting chain

Allow first child to stay on the same line with block parent

Merge pull request #1572 from topecongiro/nested-block

Nesting of block indented expressions

Add trailing comma to a single arg in multiline

Implement combining for tuple and block

Update tests

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