Skip to content

Complete nullability annotations in remaining equals methods#19016

Open
therepanic wants to merge 1 commit intospring-projects:mainfrom
therepanic:complete-nullability-annotations-in-remaining-equals-methods
Open

Complete nullability annotations in remaining equals methods#19016
therepanic wants to merge 1 commit intospring-projects:mainfrom
therepanic:complete-nullability-annotations-in-remaining-equals-methods

Conversation

@therepanic
Copy link
Copy Markdown
Contributor

In this commit, we'll finally add nullability annotations to the equals method for consistency with other projects. We've already done this, but we recently added jspecify support to new modules.

@therepanic
Copy link
Copy Markdown
Contributor Author

therepanic commented Apr 1, 2026

This PR is a follow-up to: #18930

BTW, the build crashes not because of the current changes https://github.com/spring-projects/spring-security/actions/runs/23822120186/job/69436701182/

Comment on lines 139 to 141
@Override
public boolean equals(Object obj) {
public boolean equals(@Nullable Object obj) {
if (this == obj) {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This module had jspecify support before, I guess I accidentally missed it.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 1, 2026
In this commit, we'll finally add nullability annotations to the equals
method for consistency with other projects. We've already done this, but
we recently added jspecify support to new modules.

Signed-off-by: Andrey Litvitski <andrey1010102008@gmail.com>
@therepanic therepanic force-pushed the complete-nullability-annotations-in-remaining-equals-methods branch from d2c4481 to d656397 Compare April 1, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants