Fixed

Process

Status Update

No update yet.

Description

al...@google.com

created issue #1

Nov 2, 2021 08:57PM

The following lint check is baseline suppressed in your project. Please remove all instances of this suppression from camera/camera-core/lint-baseline.xml and address the associated issues before your next stable release.

30 instance(s) of `SupportAnnotationUsage`

The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/AndroidImageProxy.java at line 101:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraControl.java at line 192:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraSelector.java at line 194:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraState.java at line 261:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraState.java at line 283:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraState.java at line 445:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraXConfig.java at line 74:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/CameraXConfig.java at line 208:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/FocusMeteringAction.java at line 159:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageAnalysis.java at line 677:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageAnalysis.java at line 725:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageAnalysis.java at line 752:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 1793:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 1810:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 1892:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 1963:
```
    @RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info
    ~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 2054:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 2077:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 2319:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageCapture.java at line 2508:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ImageProxy.java at line 89:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/Preview.java at line 564:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/Preview.java at line 654:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/impl/StateObservable.java at line 185:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/SurfaceRequest.java at line 479:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/SurfaceRequest.java at line 504:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/SurfaceRequest.java at line 658:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/UseCaseGroup.java at line 69:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/VideoCapture.java at line 1315:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```
The API level is already known to be at least 21 here from outer annotations

Found in src/main/java/androidx/camera/core/ViewPort.java at line 204:
```
@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
~~~~~~~~~~~~~~~~
```

Comments

wu...@google.com <wu...@google.com> Nov 3, 2021 02:28AM

Reassigned to ch...@google.com.

ch...@google.com <ch...@google.com> Nov 3, 2021 07:59AM

Accepted by ch...@google.com.

ch...@google.com <ch...@google.com> #2Nov 3, 2021 08:57AM

Assigned to al...@google.com.

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

al...@google.com <al...@google.com> #3Nov 3, 2021 01:53PM

Reassigned to ch...@google.com.

Great! Thanks a lot, I'll look for the live updates soon!

al...@google.com <al...@google.com> #4Nov 3, 2021 02:05PM

Upon further inspection, it looks like this issue was filed accidentally due to an issue with our baseline bug filing script.

You may still want to fix the baselined lint issues -- they are still technically incorrect -- but feel free to mark this as WAI.

ch...@google.com <ch...@google.com> #5Nov 4, 2021 07:38AM

Thanks Alan,

I will handle this issue according to which of b/204983611 and b/200306659 can be resolved first.

wu...@google.com <wu...@google.com> Nov 10, 2021 08:36AM

Accepted by ch...@google.com.

ap...@google.com <ap...@google.com> #6Nov 11, 2021 02:21AM

Project: platform/frameworks/support
Branch: androidx-main

commit 0a26ad41b1852f94276f15922706896324fed61c
Author: Charcoal Chen <charcoalchen@google.com>
Date: Wed Nov 03 14:17:55 2021

Fix SupportAnnotationUsage lint errors in CameraX proejcts

- Removed "@RequiresApi(21) // ..." annotations if it has been annotated in the outer class.
- Update api txt of the camera-core and camera-video projects

Relnote: "Removed unnecessary @RequiresApi(21) annotations from the inner classes/interfaces."

Bug: 204917951
Test: ./gradlew bOS
Change-Id: I8e28678667e2f8a5e982d64674abe4c19b5e9155

M camera/camera-testlib-extensions/src/main/java/androidx/camera/extensions/impl/HdrPreviewExtenderImpl.java
M camera/camera-video/api/public_plus_experimental_current.txt
M camera/camera-testing/src/main/java/androidx/camera/testing/CoreAppTestUtil.java
M camera/camera-testing/src/main/java/androidx/camera/testing/fakes/FakeAppConfig.java
M camera/camera-camera2-pipe-integration/src/main/java/androidx/camera/camera2/pipe/integration/impl/FocusMeteringControl.kt
M camera/camera-core/src/main/java/androidx/camera/core/impl/StateObservable.java
M camera/camera-core/src/main/java/androidx/camera/core/CameraState.java
M camera/camera-core/src/main/java/androidx/camera/core/Preview.java
M camera/camera-core/src/main/java/androidx/camera/core/ImageAnalysis.java
M camera/camera-core/src/main/java/androidx/camera/core/FocusMeteringAction.java
M camera/camera-core/src/main/java/androidx/camera/core/ImageProxy.java
M camera/camera-video/src/main/java/androidx/camera/video/MediaStoreOutputOptions.java
M camera/camera-core/src/main/java/androidx/camera/core/AndroidImageProxy.java
M camera/camera-core/src/main/java/androidx/camera/core/ViewPort.java
M camera/camera-video/src/main/java/androidx/camera/video/QualitySelector.java
M camera/camera-core/src/main/java/androidx/camera/core/VideoCapture.java
M camera/camera-core/src/main/java/androidx/camera/core/UseCaseGroup.java
M camera/camera-core/src/main/java/androidx/camera/core/CameraSelector.java
M camera/camera-video/api/current.txt
M camera/camera-core/api/current.txt
M camera/camera-core/src/main/java/androidx/camera/core/CameraXConfig.java
M camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/CameraDeviceStateCallbacks.java
M camera/camera-core/src/main/java/androidx/camera/core/CameraControl.java
M camera/camera-core/api/public_plus_experimental_current.txt
M camera/camera-core/src/main/java/androidx/camera/core/SurfaceRequest.java
M camera/camera-core/src/main/java/androidx/camera/core/ImageCapture.java
M camera/camera-core/api/restricted_current.txt
M camera/camera-extensions/src/main/java/androidx/camera/extensions/internal/sessionprocessor/SessionProcessorBase.java
M camera/camera-video/api/restricted_current.txt

https://android-review.googlesource.com/1879298

ch...@google.com <ch...@google.com> Nov 11, 2021 02:34AM

Marked as fixed.

IssueTracker

Fix 30 lint baseline suppression(s) of SupportAnnotationUsage

Status Update

Description

30 instance(s) of `SupportAnnotationUsage`

Comments

wu...@google.com <wu...@google.com> Nov 3, 2021 02:28AM

ch...@google.com <ch...@google.com> Nov 3, 2021 07:59AM

ch...@google.com <ch...@google.com> #2Nov 3, 2021 08:57AM

al...@google.com <al...@google.com> #3Nov 3, 2021 01:53PM

al...@google.com <al...@google.com> #4Nov 3, 2021 02:05PM

ch...@google.com <ch...@google.com> #5Nov 4, 2021 07:38AM

wu...@google.com <wu...@google.com> Nov 10, 2021 08:36AM

ap...@google.com <ap...@google.com> #6Nov 11, 2021 02:21AM

ch...@google.com <ch...@google.com> Nov 11, 2021 02:34AM

Issue 204917951

Description

30 instance(s) of SupportAnnotationUsage

Issue summary

Comments

wu...@google.com <wu...@google.com> Nov 3, 2021 02:28AM

ch...@google.com <ch...@google.com> Nov 3, 2021 07:59AM

ch...@google.com <ch...@google.com> #2Nov 3, 2021 08:57AM

al...@google.com <al...@google.com> #3Nov 3, 2021 01:53PM

al...@google.com <al...@google.com> #4Nov 3, 2021 02:05PM

ch...@google.com <ch...@google.com> #5Nov 4, 2021 07:38AM

wu...@google.com <wu...@google.com> Nov 10, 2021 08:36AM

ap...@google.com <ap...@google.com> #6Nov 11, 2021 02:21AM

ch...@google.com <ch...@google.com> Nov 11, 2021 02:34AM

Add comment

Issue metadata

30 instance(s) of `SupportAnnotationUsage`