Skip to content

Methods that return the result of Map.remove are not declared with a @Nullable return type#50972

Open
msridhar wants to merge 1 commit into
spring-projects:mainfrom
msridhar:map-remove-nullable
Open

Methods that return the result of Map.remove are not declared with a @Nullable return type#50972
msridhar wants to merge 1 commit into
spring-projects:mainfrom
msridhar:map-remove-nullable

Conversation

@msridhar

Copy link
Copy Markdown
Contributor

The next release of NullAway treats Map.remove as returning @Nullable, leading to errors at these locations. Fix them by marking the return type as @Nullable.

…ing @nullable

Signed-off-by: Manu Sridharan <msridhar@gmail.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 18, 2026
@wilkinsona wilkinsona changed the title Add @Nullable annotations required when treating Map.remove as returning @Nullable Methods that return the result of Map.remove are not declared as @Nullable Jul 19, 2026
@wilkinsona wilkinsona changed the title Methods that return the result of Map.remove are not declared as @Nullable Methods that return the result of Map.remove are not declared with an @Nullable return type Jul 19, 2026
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 19, 2026
@wilkinsona wilkinsona added this to the 4.0.x milestone Jul 19, 2026
@wilkinsona wilkinsona changed the title Methods that return the result of Map.remove are not declared with an @Nullable return type Methods that return the result of Map.remove are not declared with a @Nullable return type Jul 19, 2026
@msridhar

Copy link
Copy Markdown
Contributor Author

This PR should fix the build failures when updating to NullAway 0.13.8

@snicoll snicoll self-assigned this Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants