Skip to content

Commit

Permalink
Remove unneeded @SuppressWarnings("NullAway")
Browse files Browse the repository at this point in the history
Closes gh-32542
  • Loading branch information
sdeleuze committed May 6, 2024
1 parent 93dcbc4 commit 12272d6
Show file tree
Hide file tree
Showing 10 changed files with 1 addition and 11 deletions.
2 changes: 1 addition & 1 deletion gradle/spring-module.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -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'
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,6 @@ public void afterPropertiesSet() {
* @see #getResolvedDataSources()
* @see #getResolvedDefaultDataSource()
*/
@SuppressWarnings("NullAway")
public void initialize() {
if (this.targetDataSources == null) {
throw new IllegalArgumentException("Property 'targetDataSources' is required");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,6 @@ protected CompositeMessageCondition getCondition(AnnotatedElement element) {
* @param destinations the destinations
* @return new array with the processed destinations or the same array
*/
@SuppressWarnings("NullAway")
protected String[] processDestinations(String[] destinations) {
if (this.valueResolver != null) {
destinations = Arrays.stream(destinations)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -645,7 +645,6 @@ public Map<String, Object> getRawHeaders() {
return super.getRawHeaders();
}

@SuppressWarnings("NullAway")
public void setImmutable() {
if (!this.mutable) {
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,6 @@ public Mono<Void> then() {
return fetch().rowsUpdated().then();
}

@SuppressWarnings("NullAway")
private ResultFunction getResultFunction(Supplier<String> sqlSupplier) {
BiFunction<Connection, String, Statement> statementFunction = (connection, sql) -> {
if (logger.isDebugEnabled()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -374,7 +374,6 @@ public ResponseSpec exchange() {
DefaultWebTestClient.this.entityResultConsumer, getResponseTimeout());
}

@SuppressWarnings("NullAway")
private ClientRequest.Builder initRequestBuilder() {
return ClientRequest.create(this.httpMethod, initUri())
.headers(headersToUse -> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ protected ViewResultMatchers() {
/**
* Assert the selected view name with the given Hamcrest {@link Matcher}.
*/
@SuppressWarnings("NullAway")
public ResultMatcher name(Matcher<? super String> matcher) {
return result -> {
ModelAndView mav = result.getModelAndView();
Expand All @@ -61,7 +60,6 @@ public ResultMatcher name(Matcher<? super String> matcher) {
/**
* Assert the selected view name.
*/
@SuppressWarnings("NullAway")
public ResultMatcher name(String expectedViewName) {
return result -> {
ModelAndView mav = result.getModelAndView();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,6 @@ protected void hookOnSubscribe(Subscription subscription) {
}

@Override
@SuppressWarnings("NullAway")
protected void hookOnNext(DataBuffer value) {
this.requestOutstanding.set(false);
this.state.get().onNext(value);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -543,7 +543,6 @@ else if (CollectionUtils.isEmpty(defaultHeaders)) {
}
}

@SuppressWarnings("NullAway")
private ClientHttpRequest createRequest(URI uri) throws IOException {
ClientHttpRequestFactory factory;
if (DefaultRestClient.this.interceptors != null) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,6 @@ private class ConcurrentResultHandlerMethod extends ServletInvocableHandlerMetho

private final MethodParameter returnType;

@SuppressWarnings("NullAway")
public ConcurrentResultHandlerMethod(@Nullable Object result, ConcurrentResultMethodParameter returnType) {
super((Callable<Object>) () -> {
if (result instanceof Exception exception) {
Expand Down

0 comments on commit 12272d6

Please sign in to comment.