Polish "Wrap ternary operator within parentheses"
See gh-31076
This commit is contained in:
@@ -260,7 +260,7 @@ final class TestNamespaceHandler extends NamespaceHandlerSupport {
|
||||
Element element = (Element) node;
|
||||
BeanDefinition def = definition.getBeanDefinition();
|
||||
|
||||
MutablePropertyValues mpvs = (def.getPropertyValues() == null) ? new MutablePropertyValues() : def.getPropertyValues();
|
||||
MutablePropertyValues mpvs = (def.getPropertyValues() == null ? new MutablePropertyValues() : def.getPropertyValues());
|
||||
mpvs.add("name", element.getAttribute("name"));
|
||||
mpvs.add("age", element.getAttribute("age"));
|
||||
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/*
|
||||
* Copyright 2002-2022 the original author or authors.
|
||||
* Copyright 2002-2023 the original author or authors.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
@@ -63,7 +63,7 @@ public class ConcurrentMapCacheTests extends AbstractValueAdaptingCacheTests<Con
|
||||
|
||||
@Override
|
||||
protected ConcurrentMapCache getCache(boolean allowNull) {
|
||||
return allowNull ? this.cache : this.cacheNoNull;
|
||||
return (allowNull ? this.cache : this.cacheNoNull);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -128,7 +128,7 @@ public class Spr8954Tests {
|
||||
|
||||
@Override
|
||||
public Class<?> predictBeanType(Class<?> beanClass, String beanName) {
|
||||
return FactoryBean.class.isAssignableFrom(beanClass) ? PredictedType.class : null;
|
||||
return (FactoryBean.class.isAssignableFrom(beanClass) ? PredictedType.class : null);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -131,7 +131,7 @@ public class ApplicationContextEventTests extends AbstractApplicationEventListen
|
||||
else {
|
||||
smc.multicastEvent(event);
|
||||
}
|
||||
int invocation = match ? 1 : 0;
|
||||
int invocation = (match ? 1 : 0);
|
||||
verify(listener, times(invocation)).onApplicationEvent(event);
|
||||
}
|
||||
|
||||
|
||||
@@ -132,7 +132,7 @@ class ResourceBundleMessageSourceTests {
|
||||
|
||||
Locale.setDefault(expectGermanFallback ? Locale.GERMAN : Locale.CANADA);
|
||||
assertThat(ac.getMessage("code1", null, Locale.ENGLISH)).isEqualTo("message1");
|
||||
Object expected = fallbackToSystemLocale && expectGermanFallback ? "nachricht2" : "message2";
|
||||
Object expected = (fallbackToSystemLocale && expectGermanFallback ? "nachricht2" : "message2");
|
||||
assertThat(ac.getMessage("code2", null, Locale.ENGLISH)).isEqualTo(expected);
|
||||
|
||||
assertThat(ac.getMessage("code2", null, Locale.GERMAN)).isEqualTo("nachricht2");
|
||||
|
||||
@@ -41,7 +41,7 @@ public class DateTimeFormatterFactoryTests {
|
||||
private static final TimeZone NEW_YORK = TimeZone.getTimeZone("America/New_York");
|
||||
|
||||
// Ensure that we are testing against a timezone other than the default.
|
||||
private static final TimeZone TEST_TIMEZONE = ZURICH.equals(TimeZone.getDefault()) ? NEW_YORK : ZURICH;
|
||||
private static final TimeZone TEST_TIMEZONE = (ZURICH.equals(TimeZone.getDefault()) ? NEW_YORK : ZURICH);
|
||||
|
||||
|
||||
private DateTimeFormatterFactory factory = new DateTimeFormatterFactory();
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/*
|
||||
* Copyright 2002-2022 the original author or authors.
|
||||
* Copyright 2002-2023 the original author or authors.
|
||||
*
|
||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||
* you may not use this file except in compliance with the License.
|
||||
@@ -468,7 +468,7 @@ public class NotificationListenerTests extends AbstractMBeanServerTests {
|
||||
|
||||
public int getCount(String attribute) {
|
||||
Integer count = (Integer) this.attributeCounts.get(attribute);
|
||||
return (count == null) ? 0 : count;
|
||||
return (count == null ? 0 : count);
|
||||
}
|
||||
|
||||
public Object getLastHandback(String attributeName) {
|
||||
|
||||
Reference in New Issue
Block a user