Run Selection/Line fails when some lines end in commas

Describe the bug

Lines ending in commas seem to cause R: Run Selection/Line to send the wrong lines to the console.

To Reproduce

1. Create a file temp.R with the following content:

list(
    1,
    2
)

1. Place cursor on second or third line
2. Use command R: Run Selection/Line

Expected: All four lines are sent to console (if cursor was on second line), or only ‘2’ is sent to console (if cursor was on third line)

Actual: Middle two lines are sent to console:

> 1,
Error: unexpected ',' in "1,"
>     2
[1] 2

[andycraig]

Would anyone like to volunteer to try making the one-line change I suggested above? I think that would probably fix the comma issue and also change the behaviour so that parenthesised expressions split over several lines are sent to the console in full. If that works, a few unit tests would also need to be updated and then a PR could be made.

If no volunteers I will probably make the change myself at some point but hard to say when.

[brianmsm]

I don’t quite understand why this is the expected behavior. Everything inside a parenthesis, are there because their working together makes it work… in addition to the parenthesis itself. It’s a similar issue to whether or not line execution should involve pipe. It doesn’t make sense to execute without pipe, and it doesn’t make sense to execute an individual line inside a parenthesis without the function that calls it.

In the example of:

a(
    b(
        1
    )
)

Yes, all 5 lines should be executed.