Code fix: SA1118 (ParameterMustNotSpanMultipleLines)

[sharwell]

Implement a code fix for #52.

[jackwhelpton]

The following style of code used to pass SA1118 under the previous StyleCop version:

var t = new Test(
    new Test(
        new Test()),
    new Test(
        new Test()));

We do a lot of Pure DI/Poor Man's DI, and code like this gives a clear indication of the logical hierarchy of the application.

Is this change of behaviour deliberate? I'd rather not disable this rule entirely, as I do want to avoid splitting lines at an operator in the fashion demonstrated in the documentation:

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1118.md

[sharwell]

@jackwhelpton See #823