Remove unnecessary parens in ternary expression tests.
Also in a select few other spots.
diff --git a/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java b/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java
index 4886ceb..44fb31a 100644
--- a/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java
@@ -316,7 +316,7 @@
// instance may be changed by another thread
final ConfigurationInterpolator ciOld = getInterpolator();
final ConfigurationInterpolator ciNew =
- (ciOld != null) ? ciOld : new ConfigurationInterpolator();
+ ciOld != null ? ciOld : new ConfigurationInterpolator();
ciNew.registerLookups(lookups);
success = interpolator.compareAndSet(ciOld, ciNew);
} while (!success);
@@ -346,7 +346,7 @@
{
final ConfigurationInterpolator ciOld = getInterpolator();
final ConfigurationInterpolator ciNew =
- (ciOld != null) ? ciOld : new ConfigurationInterpolator();
+ ciOld != null ? ciOld : new ConfigurationInterpolator();
Lookup confLookup = findConfigurationLookup(ciNew);
if (confLookup == null)
{
@@ -380,7 +380,7 @@
{
final ConfigurationInterpolator ciOld = getInterpolator();
final ConfigurationInterpolator ciNew =
- (ciOld != null) ? ciOld : new ConfigurationInterpolator();
+ ciOld != null ? ciOld : new ConfigurationInterpolator();
ciNew.setParentInterpolator(parent);
success = interpolator.compareAndSet(ciOld, ciNew);
} while (!success);
@@ -557,7 +557,7 @@
public final Synchronizer getSynchronizer()
{
final Synchronizer sync = synchronizer;
- return (sync != null) ? sync : NoOpSynchronizer.INSTANCE;
+ return sync != null ? sync : NoOpSynchronizer.INSTANCE;
}
/**
@@ -1355,7 +1355,7 @@
public String getString(final String key, final String defaultValue)
{
final String result = convert(String.class, key, null, false);
- return (result != null) ? result : interpolate(defaultValue);
+ return result != null ? result : interpolate(defaultValue);
}
/**
@@ -1375,7 +1375,7 @@
}
final String value = getString(key);
- return (value != null) ? decoder.decode(value) : null;
+ return value != null ? decoder.decode(value) : null;
}
/**
@@ -1419,7 +1419,7 @@
public String[] getStringArray(final String key)
{
final String[] result = (String[]) getArray(String.class, key);
- return (result == null) ? new String[0] : result;
+ return result == null ? new String[0] : result;
}
/**
@@ -1561,7 +1561,7 @@
}
final Collection<T> targetCol =
- (target != null) ? target : new ArrayList<>();
+ target != null ? target : new ArrayList<>();
getConversionHandler().toCollection(src, cls, getInterpolator(),
targetCol);
return targetCol;
@@ -1701,7 +1701,7 @@
*/
protected final void initLogger(final ConfigurationLogger log)
{
- this.log = (log != null) ? log : ConfigurationLogger.newDummyLogger();
+ this.log = log != null ? log : ConfigurationLogger.newDummyLogger();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/AbstractHierarchicalConfiguration.java b/src/main/java/org/apache/commons/configuration2/AbstractHierarchicalConfiguration.java
index ac2014c..252512d 100644
--- a/src/main/java/org/apache/commons/configuration2/AbstractHierarchicalConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/AbstractHierarchicalConfiguration.java
@@ -253,7 +253,7 @@
@Override
public ExpressionEngine getExpressionEngine()
{
- return (expressionEngine != null) ? expressionEngine
+ return expressionEngine != null ? expressionEngine
: DefaultExpressionEngine.INSTANCE;
}
@@ -303,7 +303,7 @@
{
return null;
}
- return (list.size() == 1) ? list.get(0) : list;
+ return list.size() == 1 ? list.get(0) : list;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/BaseHierarchicalConfiguration.java b/src/main/java/org/apache/commons/configuration2/BaseHierarchicalConfiguration.java
index 298eee2..dc3c60e 100644
--- a/src/main/java/org/apache/commons/configuration2/BaseHierarchicalConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/BaseHierarchicalConfiguration.java
@@ -696,7 +696,7 @@
private static NodeModel<ImmutableNode> createNodeModel(
final HierarchicalConfiguration<ImmutableNode> c)
{
- final ImmutableNode root = (c != null) ? obtainRootNode(c) : null;
+ final ImmutableNode root = c != null ? obtainRootNode(c) : null;
return new InMemoryNodeModel(root);
}
@@ -836,7 +836,7 @@
// Insert all new nodes
final ImmutableNode sibling2 =
- (refHandler.getReference(nd) == null) ? null : nd;
+ refHandler.getReference(nd) == null ? null : nd;
for (final ImmutableNode insertNode : newNodes)
{
if (refHandler.getReference(insertNode) == null)
@@ -977,7 +977,7 @@
final boolean attributeChanged =
interpolateAttributes(node, handler, interpolatedAttributes);
final ImmutableNode newNode =
- (valueChanged(value, handler.getValue(node)) || attributeChanged) ? new ImmutableNode.Builder()
+ valueChanged(value, handler.getValue(node)) || attributeChanged ? new ImmutableNode.Builder()
.name(handler.nodeName(node)).value(value)
.addAttributes(interpolatedAttributes).create()
: node;
diff --git a/src/main/java/org/apache/commons/configuration2/CombinedConfiguration.java b/src/main/java/org/apache/commons/configuration2/CombinedConfiguration.java
index a46f29c..1c8c60f 100644
--- a/src/main/java/org/apache/commons/configuration2/CombinedConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/CombinedConfiguration.java
@@ -230,7 +230,7 @@
*/
public CombinedConfiguration(final NodeCombiner comb)
{
- nodeCombiner = (comb != null) ? comb : DEFAULT_COMBINER;
+ nodeCombiner = comb != null ? comb : DEFAULT_COMBINER;
initChildCollections();
}
@@ -1084,7 +1084,7 @@
public ImmutableNode getTransformedRoot()
{
final ImmutableNode configRoot = getRootNodeOfConfiguration();
- return (atPath == null) ? configRoot : prependAtPath(configRoot);
+ return atPath == null ? configRoot : prependAtPath(configRoot);
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/ConfigurationXMLReader.java b/src/main/java/org/apache/commons/configuration2/ConfigurationXMLReader.java
index 88015df..9ae0364 100644
--- a/src/main/java/org/apache/commons/configuration2/ConfigurationXMLReader.java
+++ b/src/main/java/org/apache/commons/configuration2/ConfigurationXMLReader.java
@@ -264,7 +264,7 @@
{
try
{
- final Attributes at = (attribs == null) ? EMPTY_ATTRS : attribs;
+ final Attributes at = attribs == null ? EMPTY_ATTRS : attribs;
getContentHandler().startElement(NS_URI, name, name, at);
}
catch (final SAXException ex)
diff --git a/src/main/java/org/apache/commons/configuration2/DataConfiguration.java b/src/main/java/org/apache/commons/configuration2/DataConfiguration.java
index 7caa17e..673d430 100644
--- a/src/main/java/org/apache/commons/configuration2/DataConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/DataConfiguration.java
@@ -1688,7 +1688,7 @@
private DefaultConversionHandler getOriginalConversionHandler()
{
final ConversionHandler handler = super.getConversionHandler();
- return (DefaultConversionHandler) ((handler instanceof DefaultConversionHandler) ? handler
+ return (DefaultConversionHandler) (handler instanceof DefaultConversionHandler ? handler
: null);
}
@@ -1726,7 +1726,7 @@
final DefaultConversionHandler orgHandler =
getOriginalConversionHandler();
- return (orgHandler != null) ? orgHandler.getDateFormat() : null;
+ return orgHandler != null ? orgHandler.getDateFormat() : null;
}
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/DatabaseConfiguration.java b/src/main/java/org/apache/commons/configuration2/DatabaseConfiguration.java
index b82865e..2d8554a 100644
--- a/src/main/java/org/apache/commons/configuration2/DatabaseConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/DatabaseConfiguration.java
@@ -344,7 +344,7 @@
}
if (!results.isEmpty())
{
- return (results.size() > 1) ? results : results
+ return results.size() > 1 ? results : results
.get(0);
}
return null;
@@ -673,7 +673,7 @@
private static Object convertClob(final Clob clob) throws SQLException
{
final int len = (int) clob.length();
- return (len > 0) ? clob.getSubString(1, len) : StringUtils.EMPTY;
+ return len > 0 ? clob.getSubString(1, len) : StringUtils.EMPTY;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/HierarchicalConfigurationXMLReader.java b/src/main/java/org/apache/commons/configuration2/HierarchicalConfigurationXMLReader.java
index 4037b25..e8dd560 100644
--- a/src/main/java/org/apache/commons/configuration2/HierarchicalConfigurationXMLReader.java
+++ b/src/main/java/org/apache/commons/configuration2/HierarchicalConfigurationXMLReader.java
@@ -201,7 +201,7 @@
private String nodeName(final T node, final NodeHandler<T> handler)
{
final String nodeName = handler.nodeName(node);
- return (nodeName == null) ? getRootName() : nodeName;
+ return nodeName == null ? getRootName() : nodeName;
}
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/SubnodeConfiguration.java b/src/main/java/org/apache/commons/configuration2/SubnodeConfiguration.java
index 9626d71..caa6602 100644
--- a/src/main/java/org/apache/commons/configuration2/SubnodeConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/SubnodeConfiguration.java
@@ -170,7 +170,7 @@
*/
public void close()
{
- (getTrackedModel()).close();
+ getTrackedModel().close();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/SubsetConfiguration.java b/src/main/java/org/apache/commons/configuration2/SubsetConfiguration.java
index 4edc774..925f865 100644
--- a/src/main/java/org/apache/commons/configuration2/SubsetConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/SubsetConfiguration.java
@@ -237,7 +237,7 @@
@Override
public ListDelimiterHandler getListDelimiterHandler()
{
- return (parent instanceof AbstractConfiguration) ? ((AbstractConfiguration) parent)
+ return parent instanceof AbstractConfiguration ? ((AbstractConfiguration) parent)
.getListDelimiterHandler() : super.getListDelimiterHandler();
}
diff --git a/src/main/java/org/apache/commons/configuration2/XMLPropertiesConfiguration.java b/src/main/java/org/apache/commons/configuration2/XMLPropertiesConfiguration.java
index 624b243..85152a0 100644
--- a/src/main/java/org/apache/commons/configuration2/XMLPropertiesConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/XMLPropertiesConfiguration.java
@@ -206,7 +206,7 @@
{
final PrintWriter writer = new PrintWriter(out);
- String encoding = (locator != null) ? locator.getEncoding() : null;
+ String encoding = locator != null ? locator.getEncoding() : null;
if (encoding == null)
{
encoding = DEFAULT_ENCODING;
diff --git a/src/main/java/org/apache/commons/configuration2/beanutils/BeanHelper.java b/src/main/java/org/apache/commons/configuration2/beanutils/BeanHelper.java
index 7b03664..9675c10 100644
--- a/src/main/java/org/apache/commons/configuration2/beanutils/BeanHelper.java
+++ b/src/main/java/org/apache/commons/configuration2/beanutils/BeanHelper.java
@@ -116,7 +116,7 @@
public BeanHelper(final BeanFactory defFactory)
{
defaultBeanFactory =
- (defFactory != null) ? defFactory : DefaultBeanFactory.INSTANCE;
+ defFactory != null ? defFactory : DefaultBeanFactory.INSTANCE;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/beanutils/ConfigurationDynaBean.java b/src/main/java/org/apache/commons/configuration2/beanutils/ConfigurationDynaBean.java
index d15724e..6a8075b 100644
--- a/src/main/java/org/apache/commons/configuration2/beanutils/ConfigurationDynaBean.java
+++ b/src/main/java/org/apache/commons/configuration2/beanutils/ConfigurationDynaBean.java
@@ -247,6 +247,6 @@
+ "' does not exist.");
}
- return (property instanceof List) || property.getClass().isArray();
+ return property instanceof List || property.getClass().isArray();
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/beanutils/DefaultBeanFactory.java b/src/main/java/org/apache/commons/configuration2/beanutils/DefaultBeanFactory.java
index 6d61259..54cc1b2 100644
--- a/src/main/java/org/apache/commons/configuration2/beanutils/DefaultBeanFactory.java
+++ b/src/main/java/org/apache/commons/configuration2/beanutils/DefaultBeanFactory.java
@@ -82,7 +82,7 @@
public DefaultBeanFactory(final ConversionHandler convHandler)
{
conversionHandler =
- (convHandler != null) ? convHandler
+ convHandler != null ? convHandler
: DefaultConversionHandler.INSTANCE;
}
diff --git a/src/main/java/org/apache/commons/configuration2/builder/BasicConfigurationBuilder.java b/src/main/java/org/apache/commons/configuration2/builder/BasicConfigurationBuilder.java
index 6f6a2f6..19e2b48 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/BasicConfigurationBuilder.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/BasicConfigurationBuilder.java
@@ -515,7 +515,7 @@
{
final BeanHelper helper =
BasicBuilderParameters.fetchBeanHelper(getParameters());
- return (helper != null) ? helper : BeanHelper.INSTANCE;
+ return helper != null ? helper : BeanHelper.INSTANCE;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java b/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java
index 74d9ac9..61c611e 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/BuilderConfigurationWrapperFactory.java
@@ -301,7 +301,7 @@
throws Exception
{
final Object target =
- (EventSourceSupport.DUMMY == eventSourceSupport) ? ConfigurationUtils
+ EventSourceSupport.DUMMY == eventSourceSupport ? ConfigurationUtils
.asEventSource(this, true) : builder;
return method.invoke(target, args);
}
diff --git a/src/main/java/org/apache/commons/configuration2/builder/DefaultReloadingDetectorFactory.java b/src/main/java/org/apache/commons/configuration2/builder/DefaultReloadingDetectorFactory.java
index eef4dfc..d749921 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/DefaultReloadingDetectorFactory.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/DefaultReloadingDetectorFactory.java
@@ -43,7 +43,7 @@
final Long refreshDelay = params.getReloadingRefreshDelay();
final FileHandlerReloadingDetector fileHandlerReloadingDetector =
- (refreshDelay != null) ? new FileHandlerReloadingDetector(
+ refreshDelay != null ? new FileHandlerReloadingDetector(
handler, refreshDelay) : new FileHandlerReloadingDetector(
handler);
diff --git a/src/main/java/org/apache/commons/configuration2/builder/FileBasedBuilderParametersImpl.java b/src/main/java/org/apache/commons/configuration2/builder/FileBasedBuilderParametersImpl.java
index c2e84d6..9b6c0a5 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/FileBasedBuilderParametersImpl.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/FileBasedBuilderParametersImpl.java
@@ -88,7 +88,7 @@
*/
public FileBasedBuilderParametersImpl(final FileHandler handler)
{
- fileHandler = (handler != null) ? handler : new FileHandler();
+ fileHandler = handler != null ? handler : new FileHandler();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/builder/ReloadingFileBasedConfigurationBuilder.java b/src/main/java/org/apache/commons/configuration2/builder/ReloadingFileBasedConfigurationBuilder.java
index 0901be6..de65358 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/ReloadingFileBasedConfigurationBuilder.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/ReloadingFileBasedConfigurationBuilder.java
@@ -233,7 +233,7 @@
public boolean isReloadingRequired()
{
final ReloadingDetector detector = resultReloadingDetector;
- return (detector != null) && detector.isReloadingRequired();
+ return detector != null && detector.isReloadingRequired();
}
};
}
@@ -249,6 +249,6 @@
final FileBasedBuilderParametersImpl params)
{
final ReloadingDetectorFactory factory = params.getReloadingDetectorFactory();
- return (factory != null) ? factory : DEFAULT_DETECTOR_FACTORY;
+ return factory != null ? factory : DEFAULT_DETECTOR_FACTORY;
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/builder/combined/CombinedConfigurationBuilder.java b/src/main/java/org/apache/commons/configuration2/builder/combined/CombinedConfigurationBuilder.java
index 5dad820..f13d459 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/combined/CombinedConfigurationBuilder.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/combined/CombinedConfigurationBuilder.java
@@ -1127,7 +1127,7 @@
FileBasedConfigurationBuilder fileBuilder =
(FileBasedConfigurationBuilder) defBuilder;
final URL url = fileBuilder.getFileHandler().getURL();
- currentXMLParameters.setBasePath((url != null) ? url
+ currentXMLParameters.setBasePath(url != null ? url
.toExternalForm() : fileBuilder.getFileHandler()
.getBasePath());
}
@@ -1318,7 +1318,7 @@
private Map<String, ? extends Lookup> fetchPrefixLookups()
{
final CombinedConfiguration cc = getConfigurationUnderConstruction();
- return (cc != null) ? cc.getInterpolator().getLookups() : null;
+ return cc != null ? cc.getInterpolator().getLookups() : null;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/builder/combined/FileExtensionConfigurationBuilderProvider.java b/src/main/java/org/apache/commons/configuration2/builder/combined/FileExtensionConfigurationBuilderProvider.java
index 9cf1d37..8208d2c 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/combined/FileExtensionConfigurationBuilderProvider.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/combined/FileExtensionConfigurationBuilderProvider.java
@@ -172,6 +172,6 @@
}
final int pos = fileName.lastIndexOf(EXT_SEPARATOR);
- return (pos < 0) ? null : fileName.substring(pos + 1);
+ return pos < 0 ? null : fileName.substring(pos + 1);
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/builder/fluent/Configurations.java b/src/main/java/org/apache/commons/configuration2/builder/fluent/Configurations.java
index 5c07d71..418a367 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/fluent/Configurations.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/fluent/Configurations.java
@@ -103,7 +103,7 @@
*/
public Configurations(final Parameters params)
{
- parameters = (params != null) ? params : new Parameters();
+ parameters = params != null ? params : new Parameters();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/builder/fluent/Parameters.java b/src/main/java/org/apache/commons/configuration2/builder/fluent/Parameters.java
index f34814f..cf6ee0a 100644
--- a/src/main/java/org/apache/commons/configuration2/builder/fluent/Parameters.java
+++ b/src/main/java/org/apache/commons/configuration2/builder/fluent/Parameters.java
@@ -109,7 +109,7 @@
public Parameters(final DefaultParametersManager manager)
{
defaultParametersManager =
- (manager != null) ? manager : new DefaultParametersManager();
+ manager != null ? manager : new DefaultParametersManager();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/convert/AbstractListDelimiterHandler.java b/src/main/java/org/apache/commons/configuration2/convert/AbstractListDelimiterHandler.java
index 1f304c0..89e6618 100644
--- a/src/main/java/org/apache/commons/configuration2/convert/AbstractListDelimiterHandler.java
+++ b/src/main/java/org/apache/commons/configuration2/convert/AbstractListDelimiterHandler.java
@@ -88,7 +88,7 @@
public Object escape(final Object value, final ValueTransformer transformer)
{
final Object escValue =
- (value instanceof String) ? escapeString((String) value)
+ value instanceof String ? escapeString((String) value)
: value;
return transformer.transformValue(escValue);
}
diff --git a/src/main/java/org/apache/commons/configuration2/convert/PropertyConverter.java b/src/main/java/org/apache/commons/configuration2/convert/PropertyConverter.java
index 8df939f..efd2a11 100644
--- a/src/main/java/org/apache/commons/configuration2/convert/PropertyConverter.java
+++ b/src/main/java/org/apache/commons/configuration2/convert/PropertyConverter.java
@@ -624,11 +624,11 @@
final String[] elements = ((String) value).split("_");
final int size = elements.length;
- if (size >= 1 && ((elements[0]).length() == 2 || (elements[0]).length() == 0))
+ if (size >= 1 && (elements[0].length() == 2 || elements[0].length() == 0))
{
final String language = elements[0];
- final String country = (size >= 2) ? elements[1] : "";
- final String variant = (size >= 3) ? elements[2] : "";
+ final String country = size >= 2 ? elements[1] : "";
+ final String variant = size >= 3 ? elements[2] : "";
return new Locale(language, country, variant);
}
diff --git a/src/main/java/org/apache/commons/configuration2/interpol/ConfigurationInterpolator.java b/src/main/java/org/apache/commons/configuration2/interpol/ConfigurationInterpolator.java
index 6bda7df..33d91c6 100644
--- a/src/main/java/org/apache/commons/configuration2/interpol/ConfigurationInterpolator.java
+++ b/src/main/java/org/apache/commons/configuration2/interpol/ConfigurationInterpolator.java
@@ -194,7 +194,7 @@
throw new IllegalArgumentException(
"InterpolatorSpecification must not be null!");
}
- return (spec.getInterpolator() != null) ? spec.getInterpolator()
+ return spec.getInterpolator() != null ? spec.getInterpolator()
: createInterpolator(spec);
}
diff --git a/src/main/java/org/apache/commons/configuration2/interpol/ExprLookup.java b/src/main/java/org/apache/commons/configuration2/interpol/ExprLookup.java
index 9d16a7b..bca2a5e 100644
--- a/src/main/java/org/apache/commons/configuration2/interpol/ExprLookup.java
+++ b/src/main/java/org/apache/commons/configuration2/interpol/ExprLookup.java
@@ -229,7 +229,7 @@
{
final Expression exp = engine.createExpression(result);
final Object exprResult = exp.evaluate(createContext());
- result = (exprResult != null) ? String.valueOf(exprResult) : null;
+ result = exprResult != null ? String.valueOf(exprResult) : null;
}
catch (final Exception e)
{
diff --git a/src/main/java/org/apache/commons/configuration2/io/FileHandler.java b/src/main/java/org/apache/commons/configuration2/io/FileHandler.java
index 1e177e2..9aee68e 100644
--- a/src/main/java/org/apache/commons/configuration2/io/FileHandler.java
+++ b/src/main/java/org/apache/commons/configuration2/io/FileHandler.java
@@ -393,7 +393,7 @@
{
final String fileName = file.getName();
final String basePath =
- (file.getParentFile() != null) ? file.getParentFile()
+ file.getParentFile() != null ? file.getParentFile()
.getAbsolutePath() : null;
new Updater()
{
@@ -448,7 +448,7 @@
public URL getURL()
{
final FileLocator locator = getFileLocator();
- return (locator.getSourceURL() != null) ? locator.getSourceURL()
+ return locator.getSourceURL() != null ? locator.getSourceURL()
: FileLocatorUtils.locate(locator);
}
diff --git a/src/main/java/org/apache/commons/configuration2/io/FileLocator.java b/src/main/java/org/apache/commons/configuration2/io/FileLocator.java
index 87e3077..f0d02f7 100644
--- a/src/main/java/org/apache/commons/configuration2/io/FileLocator.java
+++ b/src/main/java/org/apache/commons/configuration2/io/FileLocator.java
@@ -227,7 +227,7 @@
*/
private String sourceURLAsString()
{
- return (sourceURL != null) ? sourceURL.toExternalForm()
+ return sourceURL != null ? sourceURL.toExternalForm()
: StringUtils.EMPTY;
}
diff --git a/src/main/java/org/apache/commons/configuration2/io/FileLocatorUtils.java b/src/main/java/org/apache/commons/configuration2/io/FileLocatorUtils.java
index ed4fc9f..9ef8311 100644
--- a/src/main/java/org/apache/commons/configuration2/io/FileLocatorUtils.java
+++ b/src/main/java/org/apache/commons/configuration2/io/FileLocatorUtils.java
@@ -234,7 +234,7 @@
*/
public static boolean isLocationDefined(final FileLocator locator)
{
- return (locator != null)
+ return locator != null
&& (locator.getFileName() != null || locator.getSourceURL() != null);
}
@@ -296,7 +296,7 @@
}
final URL url = locate(locator);
- return (url != null) ? createFullyInitializedLocatorFromURL(locator,
+ return url != null ? createFullyInitializedLocatorFromURL(locator,
url) : null;
}
@@ -610,7 +610,7 @@
*/
static FileSystem obtainFileSystem(final FileLocator locator)
{
- return (locator != null) ? ObjectUtils.defaultIfNull(
+ return locator != null ? ObjectUtils.defaultIfNull(
locator.getFileSystem(), DEFAULT_FILE_SYSTEM)
: DEFAULT_FILE_SYSTEM;
}
@@ -627,7 +627,7 @@
*/
static FileLocationStrategy obtainLocationStrategy(final FileLocator locator)
{
- return (locator != null) ? ObjectUtils.defaultIfNull(
+ return locator != null ? ObjectUtils.defaultIfNull(
locator.getLocationStrategy(), DEFAULT_LOCATION_STRATEGY)
: DEFAULT_LOCATION_STRATEGY;
}
diff --git a/src/main/java/org/apache/commons/configuration2/io/FileSystem.java b/src/main/java/org/apache/commons/configuration2/io/FileSystem.java
index a696120..8aa998b 100644
--- a/src/main/java/org/apache/commons/configuration2/io/FileSystem.java
+++ b/src/main/java/org/apache/commons/configuration2/io/FileSystem.java
@@ -47,7 +47,7 @@
public ConfigurationLogger getLogger()
{
final ConfigurationLogger result = log;
- return (result != null) ? result : DEFAULT_LOG;
+ return result != null ? result : DEFAULT_LOG;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/io/HomeDirectoryLocationStrategy.java b/src/main/java/org/apache/commons/configuration2/io/HomeDirectoryLocationStrategy.java
index 27607b4..866502a 100644
--- a/src/main/java/org/apache/commons/configuration2/io/HomeDirectoryLocationStrategy.java
+++ b/src/main/java/org/apache/commons/configuration2/io/HomeDirectoryLocationStrategy.java
@@ -164,6 +164,6 @@
*/
private static String fetchHomeDirectory(final String homeDir)
{
- return (homeDir != null) ? homeDir : System.getProperty(PROP_HOME);
+ return homeDir != null ? homeDir : System.getProperty(PROP_HOME);
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/plist/PropertyListConfiguration.java b/src/main/java/org/apache/commons/configuration2/plist/PropertyListConfiguration.java
index bbe9456..d8633a8 100644
--- a/src/main/java/org/apache/commons/configuration2/plist/PropertyListConfiguration.java
+++ b/src/main/java/org/apache/commons/configuration2/plist/PropertyListConfiguration.java
@@ -523,7 +523,7 @@
protected void checkLength(final String s, final int index, final int length)
throws ParseException
{
- final int len = (s == null) ? 0 : s.length();
+ final int len = s == null ? 0 : s.length();
if (index + length > len)
{
throw new ParseException("Input string too short: " + s
diff --git a/src/main/java/org/apache/commons/configuration2/reloading/FileHandlerReloadingDetector.java b/src/main/java/org/apache/commons/configuration2/reloading/FileHandlerReloadingDetector.java
index 44c9cb3..c424823 100644
--- a/src/main/java/org/apache/commons/configuration2/reloading/FileHandlerReloadingDetector.java
+++ b/src/main/java/org/apache/commons/configuration2/reloading/FileHandlerReloadingDetector.java
@@ -85,7 +85,7 @@
*/
public FileHandlerReloadingDetector(final FileHandler handler, final long refreshDelay)
{
- fileHandler = (handler != null) ? handler : new FileHandler();
+ fileHandler = handler != null ? handler : new FileHandler();
this.refreshDelay = refreshDelay;
}
diff --git a/src/main/java/org/apache/commons/configuration2/reloading/PeriodicReloadingTrigger.java b/src/main/java/org/apache/commons/configuration2/reloading/PeriodicReloadingTrigger.java
index 624c80d..aa918d1 100644
--- a/src/main/java/org/apache/commons/configuration2/reloading/PeriodicReloadingTrigger.java
+++ b/src/main/java/org/apache/commons/configuration2/reloading/PeriodicReloadingTrigger.java
@@ -100,7 +100,7 @@
period = triggerPeriod;
timeUnit = unit;
executorService =
- (exec != null) ? exec : createDefaultExecutorService();
+ exec != null ? exec : createDefaultExecutorService();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/spring/ConfigurationPropertiesFactoryBean.java b/src/main/java/org/apache/commons/configuration2/spring/ConfigurationPropertiesFactoryBean.java
index 50d55fe..d478e6a 100644
--- a/src/main/java/org/apache/commons/configuration2/spring/ConfigurationPropertiesFactoryBean.java
+++ b/src/main/java/org/apache/commons/configuration2/spring/ConfigurationPropertiesFactoryBean.java
@@ -75,7 +75,7 @@
@Override
public Properties getObject() throws Exception
{
- return (compositeConfiguration != null) ? ConfigurationConverter.getProperties(compositeConfiguration) : null;
+ return compositeConfiguration != null ? ConfigurationConverter.getProperties(compositeConfiguration) : null;
}
/**
@@ -196,6 +196,6 @@
*/
private static <T> T[] defensiveCopy(final T[] src)
{
- return (src != null) ? src.clone() : null;
+ return src != null ? src.clone() : null;
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/sync/ReadWriteSynchronizer.java b/src/main/java/org/apache/commons/configuration2/sync/ReadWriteSynchronizer.java
index 81db5d7..0fca645 100644
--- a/src/main/java/org/apache/commons/configuration2/sync/ReadWriteSynchronizer.java
+++ b/src/main/java/org/apache/commons/configuration2/sync/ReadWriteSynchronizer.java
@@ -53,7 +53,7 @@
*/
public ReadWriteSynchronizer(final ReadWriteLock l)
{
- lock = (l != null) ? l : createDefaultLock();
+ lock = l != null ? l : createDefaultLock();
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/tree/DefaultExpressionEngine.java b/src/main/java/org/apache/commons/configuration2/tree/DefaultExpressionEngine.java
index 48dd784..67c15e0 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/DefaultExpressionEngine.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/DefaultExpressionEngine.java
@@ -158,7 +158,7 @@
symbols = syms;
nameMatcher =
- (nodeNameMatcher != null) ? nodeNameMatcher
+ nodeNameMatcher != null ? nodeNameMatcher
: NodeNameMatchers.EQUALS;
}
diff --git a/src/main/java/org/apache/commons/configuration2/tree/InMemoryNodeModel.java b/src/main/java/org/apache/commons/configuration2/tree/InMemoryNodeModel.java
index 29a0072..db28a01 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/InMemoryNodeModel.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/InMemoryNodeModel.java
@@ -725,7 +725,7 @@
final TreeData currentData = structure.get();
final InMemoryNodeModel detachedNodeModel =
currentData.getNodeTracker().getDetachedNodeModel(selector);
- return (detachedNodeModel != null) ? detachedNodeModel.getNodeHandler()
+ return detachedNodeModel != null ? detachedNodeModel.getNodeHandler()
: new TrackedNodeHandler(currentData.getNodeTracker()
.getTrackedNode(selector), currentData);
}
@@ -873,7 +873,7 @@
private TreeData createTreeData(final ImmutableNode root, final TreeData current)
{
final NodeTracker newTracker =
- (current != null) ? current.getNodeTracker()
+ current != null ? current.getNodeTracker()
.detachAllTrackedNodes() : new NodeTracker();
return createTreeDataForRootAndTracker(root, newTracker);
}
@@ -959,7 +959,7 @@
.addAttribute(addData.getNewNodeName(),
values.iterator().next()).create();
final ImmutableNode newChild =
- (pathNodeCount > 1) ? createNodeOnPath(addData
+ pathNodeCount > 1 ? createNodeOnPath(addData
.getPathNodes().subList(0, pathNodeCount - 1)
.iterator(),
Collections.singleton(childWithAttribute))
@@ -1096,7 +1096,7 @@
*/
private static ImmutableNode initialRootNode(final ImmutableNode providedRoot)
{
- return (providedRoot != null) ? providedRoot
+ return providedRoot != null ? providedRoot
: new ImmutableNode.Builder().create();
}
diff --git a/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java b/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java
index ccb2cec..d5120bd 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/ModelTransaction.java
@@ -358,7 +358,7 @@
Operations fetchOperations(final ImmutableNode target, final int level)
{
final Integer nodeLevel =
- Integer.valueOf((level == LEVEL_UNKNOWN) ? level(target)
+ Integer.valueOf(level == LEVEL_UNKNOWN ? level(target)
: level);
Map<ImmutableNode, Operations> levelOperations =
operations.get(nodeLevel);
@@ -387,7 +387,7 @@
*/
private ImmutableNode initQueryRoot(final TreeData treeData, final NodeSelector selector)
{
- return (selector == null) ? treeData.getRootNode() : treeData
+ return selector == null ? treeData.getRootNode() : treeData
.getNodeTracker().getTrackedNode(selector);
}
@@ -566,7 +566,7 @@
}
final Collection<E> result =
- (col1 != null) ? col1 : new ArrayList<>(col2.size());
+ col1 != null ? col1 : new ArrayList<>(col2.size());
result.addAll(col2);
return result;
}
@@ -586,7 +586,7 @@
return set1;
}
- final Set<E> result = (set1 != null) ? set1 : new HashSet<>();
+ final Set<E> result = set1 != null ? set1 : new HashSet<>();
result.addAll(set2);
return result;
}
@@ -608,7 +608,7 @@
return map1;
}
- final Map<K, V> result = (map1 != null) ? map1 : new HashMap<>();
+ final Map<K, V> result = map1 != null ? map1 : new HashMap<>();
result.putAll(map2);
return result;
}
@@ -624,7 +624,7 @@
*/
private static <E> Collection<E> append(final Collection<E> col, final E node)
{
- final Collection<E> result = (col != null) ? col : new LinkedList<>();
+ final Collection<E> result = col != null ? col : new LinkedList<>();
result.add(node);
return result;
}
@@ -640,7 +640,7 @@
*/
private static <E> Set<E> append(final Set<E> col, final E elem)
{
- final Set<E> result = (col != null) ? col : new HashSet<>();
+ final Set<E> result = col != null ? col : new HashSet<>();
result.add(elem);
return result;
}
@@ -658,7 +658,7 @@
*/
private static <K, V> Map<K, V> append(final Map<K, V> map, final K key, final V value)
{
- final Map<K, V> result = (map != null) ? map : new HashMap<>();
+ final Map<K, V> result = map != null ? map : new HashMap<>();
result.put(key, value);
return result;
}
@@ -810,7 +810,7 @@
*/
private Map<ImmutableNode, ImmutableNode> fetchReplacementMap()
{
- return (nodesToReplace != null) ? nodesToReplace : Collections
+ return nodesToReplace != null ? nodesToReplace : Collections
.<ImmutableNode, ImmutableNode> emptyMap();
}
@@ -822,7 +822,7 @@
*/
private Set<ImmutableNode> fetchRemovalSet()
{
- return (nodesToRemove != null) ? nodesToRemove : Collections
+ return nodesToRemove != null ? nodesToRemove : Collections
.<ImmutableNode> emptySet();
}
}
diff --git a/src/main/java/org/apache/commons/configuration2/tree/NodeSelector.java b/src/main/java/org/apache/commons/configuration2/tree/NodeSelector.java
index 9cfc89e..e4eb6bd 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/NodeSelector.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/NodeSelector.java
@@ -104,7 +104,7 @@
nodes = currentResults;
}
- return (nodes.size() == 1) ? nodes.get(0) : null;
+ return nodes.size() == 1 ? nodes.get(0) : null;
}
/**
diff --git a/src/main/java/org/apache/commons/configuration2/tree/NodeTracker.java b/src/main/java/org/apache/commons/configuration2/tree/NodeTracker.java
index c6f0c17..afbffef 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/NodeTracker.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/NodeTracker.java
@@ -486,7 +486,7 @@
*/
public ImmutableNode getNode()
{
- return (getDetachedModel() != null) ? getDetachedModel()
+ return getDetachedModel() != null ? getDetachedModel()
.getRootNode() : node;
}
@@ -532,7 +532,7 @@
*/
public TrackedNodeData observerRemoved()
{
- return (observerCount <= 1) ? null : new TrackedNodeData(node,
+ return observerCount <= 1 ? null : new TrackedNodeData(node,
observerCount - 1, getDetachedModel());
}
@@ -562,7 +562,7 @@
public TrackedNodeData detach(final ImmutableNode newNode)
{
final ImmutableNode newTrackedNode =
- (newNode != null) ? newNode : getNode();
+ newNode != null ? newNode : getNode();
return new TrackedNodeData(newTrackedNode, observerCount,
new InMemoryNodeModel(newTrackedNode));
}
diff --git a/src/main/java/org/apache/commons/configuration2/tree/OverrideCombiner.java b/src/main/java/org/apache/commons/configuration2/tree/OverrideCombiner.java
index 81076e9..61f7261 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/OverrideCombiner.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/OverrideCombiner.java
@@ -91,7 +91,7 @@
// Handle attributes and value
addAttributes(result, node1, node2);
- result.value((node1.getValue() != null) ? node1.getValue() : node2
+ result.value(node1.getValue() != null ? node1.getValue() : node2
.getValue());
return result.create();
diff --git a/src/main/java/org/apache/commons/configuration2/tree/ReferenceTracker.java b/src/main/java/org/apache/commons/configuration2/tree/ReferenceTracker.java
index b8c2faf..ddc5dab 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/ReferenceTracker.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/ReferenceTracker.java
@@ -135,7 +135,7 @@
}
List<Object> newRemovedRefs =
- (newRefs != null) ? new LinkedList<>(
+ newRefs != null ? new LinkedList<>(
removedReferences) : null;
for (final ImmutableNode node : removedNodes)
{
diff --git a/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorAttribute.java b/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorAttribute.java
index 5fd08f4..e900e20 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorAttribute.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorAttribute.java
@@ -101,7 +101,7 @@
new LinkedHashSet<>(parent.getNodeHandler()
.getAttributes(parent.getConfigurationNode()));
final String prefix =
- (name.getPrefix() != null) ? prefixName(name.getPrefix(),
+ name.getPrefix() != null ? prefixName(name.getPrefix(),
null) : null;
for (final String n : names)
{
diff --git a/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorBase.java b/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorBase.java
index 6f5116b..53cfea4 100644
--- a/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorBase.java
+++ b/src/main/java/org/apache/commons/configuration2/tree/xpath/ConfigurationNodeIteratorBase.java
@@ -221,7 +221,7 @@
*/
protected static String qualifiedName(final QName name)
{
- return (name.getPrefix() == null) ? name.getName() : prefixName(
+ return name.getPrefix() == null ? name.getName() : prefixName(
name.getPrefix(), name.getName());
}
}
diff --git a/src/test/java/org/apache/commons/configuration2/DatabaseConfigurationTestHelper.java b/src/test/java/org/apache/commons/configuration2/DatabaseConfigurationTestHelper.java
index 0787e6b..ed4d62b 100644
--- a/src/test/java/org/apache/commons/configuration2/DatabaseConfigurationTestHelper.java
+++ b/src/test/java/org/apache/commons/configuration2/DatabaseConfigurationTestHelper.java
@@ -162,7 +162,7 @@
.setTable(TABLE_MULTI)
.setConfigurationNameColumn(COL_NAME)
.setConfigurationName(
- (configName != null) ? configName : CONFIG_NAME);
+ configName != null ? configName : CONFIG_NAME);
}
/**
diff --git a/src/test/java/org/apache/commons/configuration2/MockInitialContextFactory.java b/src/test/java/org/apache/commons/configuration2/MockInitialContextFactory.java
index 7613c16..eda0941 100644
--- a/src/test/java/org/apache/commons/configuration2/MockInitialContextFactory.java
+++ b/src/test/java/org/apache/commons/configuration2/MockInitialContextFactory.java
@@ -133,7 +133,7 @@
for (int i = 0; i < PROP_NAMES.length; i++)
{
bind(mockCtx, prefix + PROP_NAMES[i], PROP_VALUES[i]);
- final String errProp = (prefix.length() > 0) ? PROP_NAMES[i] : PREFIX
+ final String errProp = prefix.length() > 0 ? PROP_NAMES[i] : PREFIX
+ PROP_NAMES[i];
bindError(mockCtx, errProp);
}
diff --git a/src/test/java/org/apache/commons/configuration2/TestAbstractHierarchicalConfiguration.java b/src/test/java/org/apache/commons/configuration2/TestAbstractHierarchicalConfiguration.java
index 74efb5e..cd3a55f 100644
--- a/src/test/java/org/apache/commons/configuration2/TestAbstractHierarchicalConfiguration.java
+++ b/src/test/java/org/apache/commons/configuration2/TestAbstractHierarchicalConfiguration.java
@@ -972,7 +972,7 @@
{
final Set<String> values = new HashSet<>();
for (final String anExpected : expected) {
- values.add((anExpected.startsWith(prefix)) ? anExpected : prefix + "." + anExpected);
+ values.add(anExpected.startsWith(prefix) ? anExpected : prefix + "." + anExpected);
}
final Iterator<String> itKeys = config.getKeys(prefix);
diff --git a/src/test/java/org/apache/commons/configuration2/TestXMLConfiguration.java b/src/test/java/org/apache/commons/configuration2/TestXMLConfiguration.java
index c8967ae..16681e4 100644
--- a/src/test/java/org/apache/commons/configuration2/TestXMLConfiguration.java
+++ b/src/test/java/org/apache/commons/configuration2/TestXMLConfiguration.java
@@ -1158,7 +1158,7 @@
{
assertEquals("Wrong value of element " + (i + 1), i,
conf.getInt("attrList.a(" + i + ")"));
- assertEquals("Wrong name attribute for element " + (i), "y",
+ assertEquals("Wrong name attribute for element " + i, "y",
conf.getString("attrList.a(" + i + ")[@name]"));
}
}
diff --git a/src/test/java/org/apache/commons/configuration2/builder/combined/TestConfigurationDeclaration.java b/src/test/java/org/apache/commons/configuration2/builder/combined/TestConfigurationDeclaration.java
index 86ff5e5..e753eca 100644
--- a/src/test/java/org/apache/commons/configuration2/builder/combined/TestConfigurationDeclaration.java
+++ b/src/test/java/org/apache/commons/configuration2/builder/combined/TestConfigurationDeclaration.java
@@ -43,7 +43,7 @@
final HierarchicalConfiguration<?> conf)
{
final HierarchicalConfiguration<?> config =
- (conf != null) ? conf : new BaseHierarchicalConfiguration();
+ conf != null ? conf : new BaseHierarchicalConfiguration();
return new ConfigurationDeclaration(null, config);
}
diff --git a/src/test/java/org/apache/commons/configuration2/builder/combined/TestReloadingCombinedConfigurationBuilderFileBased.java b/src/test/java/org/apache/commons/configuration2/builder/combined/TestReloadingCombinedConfigurationBuilderFileBased.java
index 4c96073..1f4c46a 100644
--- a/src/test/java/org/apache/commons/configuration2/builder/combined/TestReloadingCombinedConfigurationBuilderFileBased.java
+++ b/src/test/java/org/apache/commons/configuration2/builder/combined/TestReloadingCombinedConfigurationBuilderFileBased.java
@@ -135,7 +135,7 @@
*/
private File writeReloadFile(final File f, final String content) throws IOException
{
- final File file = (f != null) ? f : folder.newFile();
+ final File file = f != null ? f : folder.newFile();
writeFile(file, content);
return file;
}
diff --git a/src/test/java/org/apache/commons/configuration2/plist/TestPropertyListConfiguration.java b/src/test/java/org/apache/commons/configuration2/plist/TestPropertyListConfiguration.java
index d2a33df..b5a1185 100644
--- a/src/test/java/org/apache/commons/configuration2/plist/TestPropertyListConfiguration.java
+++ b/src/test/java/org/apache/commons/configuration2/plist/TestPropertyListConfiguration.java
@@ -214,7 +214,7 @@
@Test
public void testData()
{
- ObjectAssert.assertInstanceOf("data", (new byte[0]).getClass(), config.getProperty("data"));
+ ObjectAssert.assertInstanceOf("data", new byte[0].getClass(), config.getProperty("data"));
ArrayAssert.assertEquals("data", "foo bar".getBytes(), (byte[]) config.getProperty("data"));
}
diff --git a/src/test/java/org/apache/commons/configuration2/tree/TestInMemoryNodeModelTrackedNodes.java b/src/test/java/org/apache/commons/configuration2/tree/TestInMemoryNodeModelTrackedNodes.java
index 11b370a..3316452 100644
--- a/src/test/java/org/apache/commons/configuration2/tree/TestInMemoryNodeModelTrackedNodes.java
+++ b/src/test/java/org/apache/commons/configuration2/tree/TestInMemoryNodeModelTrackedNodes.java
@@ -586,7 +586,7 @@
for (final ImmutableNode field : nodeFields.getChildren())
{
final String expName =
- (childIndex == idx) ? NEW_FIELD : NodeStructureHelper
+ childIndex == idx ? NEW_FIELD : NodeStructureHelper
.field(1, childIndex);
checkFieldNode(field, expName);
childIndex++;
diff --git a/src/test/java/org/apache/commons/configuration2/tree/xpath/AbstractXPathTest.java b/src/test/java/org/apache/commons/configuration2/tree/xpath/AbstractXPathTest.java
index 306d1f2..394551b 100644
--- a/src/test/java/org/apache/commons/configuration2/tree/xpath/AbstractXPathTest.java
+++ b/src/test/java/org/apache/commons/configuration2/tree/xpath/AbstractXPathTest.java
@@ -146,12 +146,12 @@
{
if (level >= 0)
{
- final String prefix = (value == null) ? "" : value + ".";
+ final String prefix = value == null ? "" : value + ".";
for (int i = 1; i <= CHILD_COUNT; i++)
{
final ImmutableNode.Builder childBuilder =
new ImmutableNode.Builder();
- childBuilder.name((i % 2 == 0) ? CHILD_NAME1 : CHILD_NAME2);
+ childBuilder.name(i % 2 == 0 ? CHILD_NAME1 : CHILD_NAME2);
final String currentValue = prefix + i;
childBuilder.value(currentValue);
createLevel(childBuilder, currentValue, level - 1);