diff --git a/gradle/spring-module.gradle b/gradle/spring-module.gradle index a6f4b94a203..e41e81f83a7 100644 --- a/gradle/spring-module.gradle +++ b/gradle/spring-module.gradle @@ -13,7 +13,7 @@ dependencies { jmh 'org.openjdk.jmh:jmh-generator-annprocess:1.37' jmh 'org.openjdk.jmh:jmh-generator-bytecode:1.37' jmh 'net.sf.jopt-simple:jopt-simple' - errorprone 'com.uber.nullaway:nullaway:0.10.24' + errorprone 'com.uber.nullaway:nullaway:0.10.26' errorprone 'com.google.errorprone:error_prone_core:2.9.0' } diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/lookup/AbstractRoutingDataSource.java b/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/lookup/AbstractRoutingDataSource.java index 05065f41c35..b9d203f2023 100644 --- a/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/lookup/AbstractRoutingDataSource.java +++ b/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/lookup/AbstractRoutingDataSource.java @@ -134,7 +134,6 @@ public abstract class AbstractRoutingDataSource extends AbstractDataSource imple * @see #getResolvedDataSources() * @see #getResolvedDefaultDataSource() */ - @SuppressWarnings("NullAway") public void initialize() { if (this.targetDataSources == null) { throw new IllegalArgumentException("Property 'targetDataSources' is required"); diff --git a/spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/reactive/MessageMappingMessageHandler.java b/spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/reactive/MessageMappingMessageHandler.java index 571ef653b35..c4872d1825f 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/reactive/MessageMappingMessageHandler.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/handler/annotation/reactive/MessageMappingMessageHandler.java @@ -305,7 +305,6 @@ public class MessageMappingMessageHandler extends AbstractMethodMessageHandler sqlSupplier) { BiFunction statementFunction = (connection, sql) -> { if (logger.isDebugEnabled()) { diff --git a/spring-test/src/main/java/org/springframework/test/web/reactive/server/DefaultWebTestClient.java b/spring-test/src/main/java/org/springframework/test/web/reactive/server/DefaultWebTestClient.java index 152a207855c..da784ae0faa 100644 --- a/spring-test/src/main/java/org/springframework/test/web/reactive/server/DefaultWebTestClient.java +++ b/spring-test/src/main/java/org/springframework/test/web/reactive/server/DefaultWebTestClient.java @@ -374,7 +374,6 @@ class DefaultWebTestClient implements WebTestClient { DefaultWebTestClient.this.entityResultConsumer, getResponseTimeout()); } - @SuppressWarnings("NullAway") private ClientRequest.Builder initRequestBuilder() { return ClientRequest.create(this.httpMethod, initUri()) .headers(headersToUse -> { diff --git a/spring-test/src/main/java/org/springframework/test/web/servlet/result/ViewResultMatchers.java b/spring-test/src/main/java/org/springframework/test/web/servlet/result/ViewResultMatchers.java index c6ac6441262..af88cb202e7 100644 --- a/spring-test/src/main/java/org/springframework/test/web/servlet/result/ViewResultMatchers.java +++ b/spring-test/src/main/java/org/springframework/test/web/servlet/result/ViewResultMatchers.java @@ -47,7 +47,6 @@ public class ViewResultMatchers { /** * Assert the selected view name with the given Hamcrest {@link Matcher}. */ - @SuppressWarnings("NullAway") public ResultMatcher name(Matcher matcher) { return result -> { ModelAndView mav = result.getModelAndView(); @@ -61,7 +60,6 @@ public class ViewResultMatchers { /** * Assert the selected view name. */ - @SuppressWarnings("NullAway") public ResultMatcher name(String expectedViewName) { return result -> { ModelAndView mav = result.getModelAndView(); diff --git a/spring-web/src/main/java/org/springframework/http/codec/multipart/MultipartParser.java b/spring-web/src/main/java/org/springframework/http/codec/multipart/MultipartParser.java index 00e35677388..9eb8bff26a4 100644 --- a/spring-web/src/main/java/org/springframework/http/codec/multipart/MultipartParser.java +++ b/spring-web/src/main/java/org/springframework/http/codec/multipart/MultipartParser.java @@ -116,7 +116,6 @@ final class MultipartParser extends BaseSubscriber { } @Override - @SuppressWarnings("NullAway") protected void hookOnNext(DataBuffer value) { this.requestOutstanding.set(false); this.state.get().onNext(value); diff --git a/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java b/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java index 98b2de69ae5..471f5ced20e 100644 --- a/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java +++ b/spring-web/src/main/java/org/springframework/web/client/DefaultRestClient.java @@ -543,7 +543,6 @@ final class DefaultRestClient implements RestClient { } } - @SuppressWarnings("NullAway") private ClientHttpRequest createRequest(URI uri) throws IOException { ClientHttpRequestFactory factory; if (DefaultRestClient.this.interceptors != null) { diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ServletInvocableHandlerMethod.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ServletInvocableHandlerMethod.java index b5372321d3e..4e44175032a 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ServletInvocableHandlerMethod.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ServletInvocableHandlerMethod.java @@ -215,7 +215,6 @@ public class ServletInvocableHandlerMethod extends InvocableHandlerMethod { private final MethodParameter returnType; - @SuppressWarnings("NullAway") public ConcurrentResultHandlerMethod(@Nullable Object result, ConcurrentResultMethodParameter returnType) { super((Callable) () -> { if (result instanceof Exception exception) {