Infeasible
Status Update
Comments
al...@google.com <al...@google.com> #2
I've added additional integration tests for annotation-experimental-lint and they all pass. I think this is an issue with Android Studio.
al...@google.com <al...@google.com>
ap...@google.com <ap...@google.com> #3
Project: platform/frameworks/support
Branch: androidx-master-dev
commit 96cb8bf2726eea3281166c155221f43713eff14f
Author: Alan Viverette <alanv@google.com>
Date: Fri Sep 06 17:03:28 2019
Additional regression tests for static and inline usage
Bug: 140637106
Test: UseKtExperimentalFromJava
Change-Id: I1282c051b9b722bf288b66fd8406d703fc3e29a4
A annotation/annotation-experimental-lint/integration-tests/src/main/java/sample/TimeProviderKt.kt
M annotation/annotation-experimental-lint/integration-tests/src/main/java/sample/UseKtExperimentalFromJava.java
M annotation/annotation-experimental-lint/src/test/kotlin/androidx/annotation/experimental/lint/ExperimentalDetectorTest.kt
https://android-review.googlesource.com/1116455
Branch: androidx-master-dev
commit 96cb8bf2726eea3281166c155221f43713eff14f
Author: Alan Viverette <alanv@google.com>
Date: Fri Sep 06 17:03:28 2019
Additional regression tests for static and inline usage
Bug: 140637106
Test: UseKtExperimentalFromJava
Change-Id: I1282c051b9b722bf288b66fd8406d703fc3e29a4
A annotation/annotation-experimental-lint/integration-tests/src/main/java/sample/TimeProviderKt.kt
M annotation/annotation-experimental-lint/integration-tests/src/main/java/sample/UseKtExperimentalFromJava.java
M annotation/annotation-experimental-lint/src/test/kotlin/androidx/annotation/experimental/lint/ExperimentalDetectorTest.kt
al...@google.com <al...@google.com>
bu...@google.com <bu...@google.com> #4
Hi. I've received your bug and will wait for b/140640322 to be resolved and then assign the bug to alanv@google.com.
Bugjuggler: b/140640322 -> assigned to alanv@google.com
Bugjuggler:
Description
Here's a sample project:
In the sample usages of experimental APIs in ExperimentalUsage class
public void main() {
// Should raise a lint warning?
new ExperimentalApi().experimentalFn();
// This correctly raises lint error.
//noinspection KotlinInternalInJava
BenchmarkState.Companion.reportData(null, null, 0, new ArrayList<Long>(), 0, 0, 0);
// Should raise a lint warning?
BenchmarkState.reportData(null, null, 0, new ArrayList<Long>(), 0, 0, 0);
// Should raise a lint warning?
new ExperimentalApi().experimentalJavaFn();
}
Only 1 / 4 of these correctly warn the user of experimental api usage