Reduce line count

[tcharding]

This PR is formatting changes only, totally non-urgent.

Reduce line count by setting a couple of rustfmt config options to be the same as they are in rust-bitcoin. Final patch removes whitespace in trait impl blocks, and was the impetus for this PR.

[apoelstra]

I think increasing chain_width is mostly a bad change.

[apoelstra]

But 0c7a972 looks good otherwise.

[tcharding]

I split each config option change into a separate patch so they can be viewed easily. I actually think the only massive win is fn_single_line and maybe struct_lit_width.

[tcharding]

FTR docs say the values are a percentage of max_width but anything over 80 causes an error, so there is a bug somewhere.

fn_call_width cannot have a value that exceeds max_width. fn_call_width will be set to the same value as max_width

[apoelstra]

Not really a fan of the array_width change either.

Otherwise 75d3fa8 looks good.

[tcharding]

Not really a fan of the array_width change either.

No worries, I'll drop that commit - that is exactly why I kept them all separate :)

[tcharding]

Force push removes the array_width change (was the last patch of the series).

[tcharding]

Otherwise 75d3fa8 looks good.

Just in case you go looking for it, this is the commit hash of the removed patch.

[apoelstra]

ACK 1cab0b8

[tcharding]

Rebased, no other changes.

[apoelstra]

ACK f41a70d

[tcharding]

Re-done after planning module merging. @TheBlueMatt this one should make you all warm and fuzzy inside :)

[sanket1729]

@tcharding, what order do you prefer I merge the PRs?

[apoelstra]

ACK 26bb52e

[tcharding]

This one first please, it will cause loads of conflicts but that is always going to be the case whenever it merges. I meant "first" as in out of my PRs, do other peoples before mine please, I'm happy to rebase.