Skip to content

Commit 090e394

Browse files
izeyejhoeller
authored andcommitted
Clean up @SuppressWarnings("ConstantConditions")
1 parent 708e61a commit 090e394

File tree

3 files changed

+3
-7
lines changed

3 files changed

+3
-7
lines changed

spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/support/DefaultMessageHandlerMethodFactory.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,7 @@ public void setCustomArgumentResolvers(List<HandlerMethodArgumentResolver> custo
120120
* the ones configured by default. This is an advanced option. For most use cases
121121
* it should be sufficient to use {@link #setCustomArgumentResolvers(java.util.List)}.
122122
*/
123-
@SuppressWarnings("ConstantConditions")
124-
public void setArgumentResolvers(List<HandlerMethodArgumentResolver> argumentResolvers) {
123+
public void setArgumentResolvers(@Nullable List<HandlerMethodArgumentResolver> argumentResolvers) {
125124
if (argumentResolvers == null) {
126125
this.argumentResolvers.clear();
127126
return;

spring-messaging/src/main/java/org/springframework/messaging/rsocket/annotation/support/RSocketFrameTypeMessageCondition.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,6 @@ protected String getToStringInfix() {
131131
* @param message the current message
132132
* @return the frame type or {@code null} if not found
133133
*/
134-
@SuppressWarnings("ConstantConditions")
135134
@Nullable
136135
public static FrameType getFrameType(Message<?> message) {
137136
return (FrameType) message.getHeaders().get(RSocketFrameTypeMessageCondition.FRAME_TYPE_HEADER);

spring-web/src/main/java/org/springframework/web/cors/CorsConfiguration.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -156,8 +156,7 @@ public List<String> getAllowedOrigins() {
156156
/**
157157
* Variant of {@link #setAllowedOrigins} for adding one origin at a time.
158158
*/
159-
@SuppressWarnings("ConstantConditions")
160-
public void addAllowedOrigin(String origin) {
159+
public void addAllowedOrigin(@Nullable String origin) {
161160
if (origin == null) {
162161
return;
163162
}
@@ -222,8 +221,7 @@ public List<String> getAllowedOriginPatterns() {
222221
* Variant of {@link #setAllowedOriginPatterns} for adding one origin at a time.
223222
* @since 5.3
224223
*/
225-
@SuppressWarnings("ConstantConditions")
226-
public void addAllowedOriginPattern(String originPattern) {
224+
public void addAllowedOriginPattern(@Nullable String originPattern) {
227225
if (originPattern == null) {
228226
return;
229227
}

0 commit comments

Comments
 (0)