Skip to content

Fix crash in PsiResolutionStrategy for annotations with nullable fqn#2864

Open
jaschdoc wants to merge 4 commits intomainfrom
add-test-for-local-classes
Open

Fix crash in PsiResolutionStrategy for annotations with nullable fqn#2864
jaschdoc wants to merge 4 commits intomainfrom
add-test-for-local-classes

Conversation

@jaschdoc
Copy link
Copy Markdown
Collaborator

@jaschdoc jaschdoc commented Apr 14, 2026

This PR fixes a crash in PsiResolutionStrategy if an annotation name has a nullable qualified name.

Related to #2816

@jaschdoc jaschdoc force-pushed the add-test-for-local-classes branch from 8344480 to 64fdf08 Compare April 14, 2026 11:48
@jaschdoc jaschdoc changed the title Add test for local classes Add tests for local classes Apr 14, 2026
@jaschdoc jaschdoc marked this pull request as ready for review April 14, 2026 12:25
@jaschdoc jaschdoc requested review from bcorso and troelsbjerre April 14, 2026 12:25
Copy link
Copy Markdown
Collaborator

@bcorso bcorso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jaschdoc jaschdoc force-pushed the add-test-for-local-classes branch 2 times, most recently from 14e45cc to 99e9154 Compare April 15, 2026 10:00
@jaschdoc jaschdoc changed the title Add tests for local classes Fix crash in PsiResolutionStrategy for annotations with nullable fqn Apr 15, 2026
@jaschdoc jaschdoc force-pushed the add-test-for-local-classes branch from 99e9154 to 61b9b65 Compare April 16, 2026 08:47
@jaschdoc jaschdoc force-pushed the add-test-for-local-classes branch from 61b9b65 to b4f49b5 Compare April 16, 2026 08:52
Comment thread kotlin-analysis-api/src/main/kotlin/com/google/devtools/ksp/common/MapUtils.kt Outdated
Comment thread kotlin-analysis-api/src/main/kotlin/com/google/devtools/ksp/common/MapUtils.kt Outdated
@jaschdoc jaschdoc force-pushed the add-test-for-local-classes branch from b4f49b5 to 4d5dc79 Compare April 16, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants