Skip to content

GH-19510 Simplify null check in ProviderManager using Assert.noNullElements - #19511

Open
wsdf25867 wants to merge 1 commit into
spring-projects:mainfrom
wsdf25867:gh-19510-providermanager-null-check
Open

GH-19510 Simplify null check in ProviderManager using Assert.noNullElements#19511
wsdf25867 wants to merge 1 commit into
spring-projects:mainfrom
wsdf25867:gh-19510-providermanager-null-check

Conversation

@wsdf25867

Copy link
Copy Markdown

Closes #19510

Description

Refactored ProviderManager.checkState() to use Assert.noNullElements instead of Assert.isTrue(!CollectionUtils.contains(...)).

Benefits

  • Improves code readability by eliminating double negation logic.
  • Eliminates unnecessary Iterator object creation.
  • Utilizes updated Spring Framework Assert API.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 7, 2026
…Assert.noNullElements

Signed-off-by: Jiwoo Kang <wsdf25867@gmail.com>
@wsdf25867
wsdf25867 force-pushed the gh-19510-providermanager-null-check branch from 5d8b208 to 3f14ad2 Compare August 7, 2026 02:24
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.

Improve readability and simplify null check in ProviderManager.checkState()

2 participants