Status Update
Comments
js...@google.com <js...@google.com>
js...@google.com <js...@google.com> #3
Thank you for your patience while our engineering team worked to resolve this issue. A fix for this issue is now available in:
- Android Studio Ladybug Feature Drop | 2024.2.2 Canary 6
- Android Gradle Plugin 8.8.0-alpha06
We encourage you to try the latest update.
If you notice further issues or have questions, please file a new bug report.
Thank you for taking the time to submit feedback — we really appreciate it!
js...@google.com <js...@google.com> #4
Actually, if I changed the test to include something like:
i.flags = MyIntent.UNRELATED or MyIntent.UNRELATED // ERROR 6
i.flags = MyIntent.UNRELATED.or(MyIntent.UNRELATED) // ERROR 7
it detects something like:
src/test.kt:15: Error: Must be one or more of: MyIntent.FLAG_1, FLAG_2 [WrongConstant]
i.flags = MyIntent.UNRELATED or MyIntent.UNRELATED // ERROR 6
~~~~~~~~~~~~~~~~~~
src/test.kt:15: Error: Must be one or more of: MyIntent.FLAG_1, FLAG_2 [WrongConstant]
i.flags = MyIntent.UNRELATED or MyIntent.UNRELATED // ERROR 6
~~~~~~~~~~~~~~~~~~
but complains that the same incident was reported more than once on X.or(Y)
.
It's still true that the detector should be able to detect both offending operands, but the reports should be on those operands too, not on the surrounding call.
js...@google.com <js...@google.com> #5
Oh my, this is shamefully simple bug. :facepalm: The fix will be coming shortly.
js...@google.com <js...@google.com>
an...@google.com <an...@google.com> #6
Thank you for your patience while our engineering team worked to resolve this issue. A fix for this issue is now available in:
- Android Studio Ladybug Feature Drop | 2024.2.2 Canary 6
- Android Gradle Plugin 8.8.0-alpha06
We encourage you to try the latest update.
If you notice further issues or have questions, please file a new bug report.
Thank you for taking the time to submit feedback — we really appreciate it!
ju...@google.com <ju...@google.com> #7
I'm updating to canary 6, and the line being flagged has changed, but I'm still seeing an
?.or(android.R.attr.showWhenLocked or android.R.attr.turnScreenOn)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Might have to do with the line using both .or
and infix or
?
js...@google.com <js...@google.com>
js...@google.com <js...@google.com> #8
Thank you for continued reports. :)
Might have to do with the line using both
.or
and infixor
?
It turns out that, in this case, operands of polyadic expression (super set of binary expression) need to use their own location, instead of the containing expression.
Also, thanks to this, found that the identical locations are used for if
/else
, which also assumed it's elvis operator, indicating ?:
is never handled/tested. Will be fixed together.
js...@google.com <js...@google.com>
an...@google.com <an...@google.com> #9
Further fixes for this issue are now available in:
- Android Studio Ladybug Feature Drop | 2024.2.2 Canary 8
- Android Gradle Plugin 8.8.0-alpha08
We encourage you to try the latest update.
If you notice further issues or have questions, please file a new bug report.
an...@google.com <an...@google.com> #10
The fixes for this issue are now also available in:
- Android Studio Meerkat | 2024.3.1 Canary 3
- Android Gradle Plugin 8.9.0-alpha03
We encourage you to try the latest update.
If you notice further issues or have questions, please file a new bug report.
Description
With the update to AGP 8.8.0-alpha04, the same one line in added a second time .
WrongConstant
lint error is appearing twice for:window:window-demos:demo
. The issue was already present in the baseline file, but the lint task fails unless it is