Fix unnecessary_parenthesis with null aware expressions.

[asashour]

Fixes dart-lang/sdk#59021

Description

unnecessary_parenthesis should check for an expression being null aware recursively, through the target.

[coveralls]

Coverage: 95.66% (+0.004%) from 95.656% when pulling 74c4b44 on asashour:4028_unnecessary_parenthesis into 6006d74 on dart-lang:main.

[oprypin]

Thanks. This definitely fixes the false positive.

But I noticed this also adds a new caught case. Which definitely seems to also be an improvement, but just thought I'd point it out because I'm not sure if it was expected.

Wasn't reported before, now is:

var x = (foo?.a.b.c.length) ?? 0;

[asashour]

I think the new behavior is correct, the parenthesis are unnecessary, and they can be safely removed, or I am missing something.

class A {
  A a = A();
  A b = A();
  String c = '';
}

void f(A? foo) {
  int x = (foo?.a.b.c.length) ?? 0;
}

[oprypin]

Sure, can be removed.

definitely seems to also be an improvement, but just thought I'd point it out