Shift multiline paren contents less aggressively

[brianrourkeboll]

Another followup to #16079.

Description

The logic in #16666 addressed some multiline parentheses removal issues, but it was slightly too aggressive and sometimes shifted lines farther than was necessary or safe.

For example, before #16666, the following code —

let longVarName1 = 1
let longVarName2 = 2
(
    longFunctionName
        longVarName1
        longVarName2
)

— when the unnecessary parentheses code fix was applied, would become —

let longVarName1 = 1
let longVarName2 = 2

    longFunctionName
        longVarName1
        longVarName2

— which was incorrect.

After #16666, it would become —

let longVarName1 = 1
let longVarName2 = 2
longFunctionName
longVarName1
longVarName2

— which was more correct in one way (longFunctionName needed to be shifted left), but less correct in another (the arguments to longFunctionName were shifted too far).

With this PR, it now correctly becomes:

let longVarName1 = 1
let longVarName2 = 2
longFunctionName
    longVarName1
    longVarName2

Checklist

• Test cases added.
• Release notes entry updated.

[github-actions]

❗ Release notes required

---

✅ Found changes and release notes in following paths:

Change path	Release notes path	Description
vsintegration/src	docs/release-notes/.VisualStudio/17.10.md

[psfinaki]

Thanks!

[psfinaki]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[vzarytovskii]

Thanks!