Fixed
Status Update
Comments
bl...@google.com <bl...@google.com>
bl...@google.com <bl...@google.com>
an...@google.com <an...@google.com> #2
This was broken by 3bca759a5ff08352de831bb1e9b61b1ec2b3362d.
Fix (pending) is I2c2dc7b600603ee430fd0d91b23d52ea8aa29ca9.
Fix (pending) is I2c2dc7b600603ee430fd0d91b23d52ea8aa29ca9.
mo...@google.com <mo...@google.com> #3
Almost 2 months later and this is still broken
Description
It seems NewApi linter does not understand if I assign
isAtleastU() && otherCondition()
to theprivate final boolean foo
in the constructor and haveif(foo) {//call U API}
.Linter was happy if I use local variable.
Actual code I hit this issue is that NewApi linter was not happy with patchset 16 but happy with patchset 17.
https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2764846/16..17/service/src/com/android/server/ConnectivityService.java
Linter was not happy with(patchset 16)
But Linter was happy with(patchset 17)
Also, Linter was happy with