Commit 19dc506
committed
Kotlin: Fix findTopLevelPropertyOrWarn for K2
`Receiver::class.java` resolves differently from K1:
K1: getJavaClass(KClass) - value argument
K2: KClass.<get-java>() - property receiver
This caused the lookup to fail, so
`Intent(context, Receiver::class.java)` dropped the second argument.
log:
"Couldn't find JVM intrinsic property kotlin.jvm java in kotlin.jvm.JvmClassMappingKt"
This caused false positives in `java/android/implicit-pendingintents`
The fix mirrors 5c36e63
Fixes #20153
Assisted-by: Claude Opus 4.8 - diagnostics & initial fix
Assisted-by: Claude Fable 5.1 - convert to 'when'; re-review1 parent 401a516 commit 19dc506
8 files changed
Lines changed: 44 additions & 1 deletion
File tree
- java
- kotlin-extractor/src/main/kotlin
- ql
- lib/change-notes
- test-kotlin1/library-tests/kclass-java-arg
- test-kotlin2/library-tests/kclass-java-arg
Lines changed: 8 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3905 | 3905 | | |
3906 | 3906 | | |
3907 | 3907 | | |
3908 | | - | |
| 3908 | + | |
| 3909 | + | |
| 3910 | + | |
| 3911 | + | |
| 3912 | + | |
| 3913 | + | |
| 3914 | + | |
| 3915 | + | |
3909 | 3916 | | |
3910 | 3917 | | |
3911 | 3918 | | |
| |||
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
Lines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
Lines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
0 commit comments