ResourceType lint check is not working for kotlin sources [247025215]

Fixed

Bug

Status Update

No update yet.

Description

ga...@linecorp.com

created issue #1

Sep 16, 2022 02:04AM

DESCRIBE THE ISSUE IN DETAIL:
I'm trying to update minSdk to 26,
and want to fix issues related with behavior change of resource compiler on my app.

For java source codes, lint checker raise ResourceType error,
but for kotlin source codes, lint checker did not raise any error.

STEPS TO REPRODUCE:

clone ResourceType_lint_checker branch from https://github.com/ganadist/VersionCodeDemo

git clone https://github.com/ganadist/VersionCodeDemo -b ResourceType_lint_checker

perform lint checker on command line

cd VersionCodeDemo
./gradlew :app:lintDevelopDebug

Check lint result that lint raise error for TextViewUtil1.java file only, and did not raise error for TextViewUtil2.kt file

app/src/main/java/com/example/myapplication/TextViewUtil1.java:8: Error: Comparing resource types (@StringRes) other than equality is dangerous and usually wrong;  some resource types set top bit which turns the value negative [ResourceType]
      if (resId > 0) {
          ~~~~~~~~~

 Explanation for issues of type "ResourceType":
 Ensures that resource id's passed to APIs are of the right type; for
 example, calling Resources.getColor(R.string.name) is wrong.

ATTACH SCREENSHOTS/RECORDINGS OF THE ISSUE

ATTACH LOG FILES (Select Help > Show Log in Files, or Show Log in Finder on a Mac)

IMPORTANT: Please read https://developer.android.com/studio/report-bugs.html carefully and supply all required information.

Studio Build: Any
Version of Gradle Plugin: 7.2.2 or 7.3.0
Version of Gradle: 7.5
Version of Java: OpenJDK 11 / 17
OS: Any

Comments

dy...@google.com <dy...@google.com> Sep 16, 2022 02:09AM

Assigned to an...@google.com.

ba...@google.com <ba...@google.com> Sep 19, 2022 03:43AM

Reassigned to ba...@google.com.

ba...@google.com <ba...@google.com> Sep 20, 2022 11:19PM

Reassigned to bl...@google.com.

bl...@google.com <bl...@google.com> Sep 21, 2022 10:56PM

Accepted by bl...@google.com.

bl...@google.com <bl...@google.com> Sep 22, 2022 09:35PM

Marked as fixed.

ga...@linecorp.com <ga...@linecorp.com> #2Sep 22, 2022 11:00PM

Lint options:

    lintOptions {
        lintConfig = file('lint.xml')

        abortOnError true
        xmlReport true
        htmlReport true
        checkDependencies true
    }

Also, our project has custom lint rules. Failure happens with a dependency of 26.6.3 and 27.0.0.

Is any way to downgrade lint runner? This problem is keeping us from using AGP 4.0

bl...@google.com <bl...@google.com> #3Sep 22, 2022 11:47PM

Attached slightly different stacktrace for the same problem.

Wonder if disabling some lint rules could help to "fix" this problem, as it is also blocking for us 4.0 AGP upgrade.

ga...@linecorp.com <ga...@linecorp.com> #4Sep 23, 2022 08:49AM

Notes:

I suspect the exceptions are happening now because JetBrains recently changed an assert statement into a hard failure (see 3260e41111).

I.e., it's possible that Lint has been hitting this code path all along, and only now it is causing a problem.

This has been difficult to investigate so far because the failure is nondeterministic and also deep inside the Kotlin compiler. Plus, JetBrains seems to be hitting this code path occasionally too (hence why they changed the assert into a hard failure), so I'm not even sure whether Lint is implicated or not.