You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by co...@apache.org on 2017/02/15 11:35:56 UTC
[3/3] cxf git commit: Replace .size() > 0 with !isEmpty() for
collections
Replace .size() > 0 with !isEmpty() for collections
Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/003fb1e7
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/003fb1e7
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/003fb1e7
Branch: refs/heads/master
Commit: 003fb1e7d2e5790ee44b110da83e1e97ed7d0ec5
Parents: 4301392
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Wed Feb 15 10:55:55 2017 +0000
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Wed Feb 15 10:55:55 2017 +0000
----------------------------------------------------------------------
.../org/apache/cxf/attachment/AttachmentUtil.java | 4 ++--
.../cxf/bus/extension/ExtensionManagerImpl.java | 5 -----
.../org/apache/cxf/common/jaxb/JAXBUtils.java | 5 ++---
.../stax/StaxDataBindingInterceptor.java | 2 +-
.../cxf/interceptor/ClientFaultConverter.java | 2 +-
.../java/org/apache/cxf/message/MessageImpl.java | 2 +-
.../apache/cxf/service/model/OperationInfo.java | 2 +-
.../apache/cxf/staxutils/W3CDOMStreamWriter.java | 2 +-
.../staxutils/transform/OutTransformWriter.java | 2 +-
.../AbstractBeanValidationInterceptor.java | 2 +-
.../binding/corba/runtime/CorbaObjectReader.java | 2 +-
.../binding/corba/runtime/CorbaObjectWriter.java | 2 +-
.../cxf/binding/soap/SoapBindingFactory.java | 4 ++--
.../org/apache/cxf/binding/soap/SoapFault.java | 2 +-
.../soap/interceptor/SoapActionInInterceptor.java | 4 ++--
.../soap/interceptor/SoapHeaderInterceptor.java | 2 +-
.../jms/interceptor/SoapJMSInInterceptor.java | 18 +++++++++---------
.../cxf/aegis/databinding/AegisDatabinding.java | 2 +-
.../org/apache/cxf/aegis/type/XMLTypeCreator.java | 2 +-
.../cxf/aegis/util/stax/JDOMStreamReader.java | 2 +-
.../AbstractStaticFailoverStrategy.java | 4 ++--
.../cxf/jaxrs/AbstractJAXRSFactoryBean.java | 2 +-
.../apache/cxf/jaxrs/ext/ProtocolHeadersImpl.java | 2 +-
.../cxf/jaxrs/ext/multipart/MultipartBody.java | 2 +-
.../org/apache/cxf/jaxrs/impl/RequestImpl.java | 2 +-
.../cxf/jaxrs/impl/VariantListBuilderImpl.java | 12 ++++++------
.../cxf/jaxrs/provider/AbstractJAXBProvider.java | 2 +-
.../jaxrs/provider/RequestDispatcherProvider.java | 2 +-
.../cxf/jaxrs/provider/ServerProviderFactory.java | 2 +-
.../cxf/jaxrs/provider/XSLTJaxbProvider.java | 2 +-
.../jaxrs/security/JAASAuthenticationFilter.java | 2 +-
.../jaxrs/servlet/CXFNonSpringJaxrsServlet.java | 4 ++--
.../apache/cxf/jaxrs/utils/InjectionUtils.java | 4 ++--
.../org/apache/cxf/jaxrs/utils/JAXRSUtils.java | 4 ++--
.../jaxrs/utils/multipart/AttachmentUtils.java | 2 +-
.../java/org/apache/cxf/jaxws/EndpointImpl.java | 2 +-
.../org/apache/cxf/jaxws/spi/ProviderImpl.java | 2 +-
.../cxf/jaxws/support/JaxWsImplementorInfo.java | 4 ++--
.../jaxws/support/JaxWsServiceFactoryBean.java | 2 +-
.../endpoint/dynamic/DynamicClientFactory.java | 2 +-
.../apache/cxf/javascript/JsSimpleDomNode.java | 2 +-
.../cxf/jaxrs/model/wadl/WadlGeneratorTest.java | 2 +-
.../jaxrs/rx/server/StreamingAsyncSubscriber.java | 2 +-
.../apache/cxf/jaxrs/ext/search/SearchUtils.java | 2 +-
.../oauth/filters/AbstractAuthFilter.java | 2 +-
.../http_jetty/JettyHTTPServerEngine.java | 2 +-
...TPServerEngineFactoryBeanDefinitionParser.java | 2 +-
...tpServerEngineFactoryBeanDefinitionParser.java | 2 +-
...TPServerEngineFactoryBeanDefinitionParser.java | 2 +-
.../org/apache/cxf/transport/http/Cookies.java | 2 +-
.../apache/cxf/transport/http/HTTPConduit.java | 4 ++--
.../apache/cxf/transport/jms/JMSMessageUtils.java | 2 +-
.../cxf/ws/addressing/impl/MAPAggregatorImpl.java | 2 +-
.../main/java/org/apache/cxf/ws/rm/RMManager.java | 2 +-
.../org/apache/cxf/ws/rm/SequenceMonitor.java | 2 +-
.../cxf/ws/rm/persistence/jdbc/RMTxStore.java | 2 +-
.../cxf/ws/rm/soap/RMSoapInInterceptor.java | 4 ++--
.../cxf/ws/rm/soap/RedeliveryQueueImpl.java | 2 +-
.../cxf/ws/rm/soap/RetransmissionQueueImpl.java | 2 +-
.../IssuedTokenInterceptorProvider.java | 2 +-
.../KerberosTokenInterceptorProvider.java | 2 +-
.../wss4j/PolicyBasedWSS4JOutInterceptor.java | 2 +-
.../ws/security/wss4j/WSS4JOutInterceptor.java | 2 +-
.../policyhandlers/AbstractBindingBuilder.java | 2 +-
.../policyhandlers/AsymmetricBindingHandler.java | 6 +++---
.../StaxAsymmetricBindingHandler.java | 8 ++++----
.../StaxSymmetricBindingHandler.java | 4 ++--
.../policyhandlers/SymmetricBindingHandler.java | 6 +++---
.../cxf/wsdl/interceptors/BareInInterceptor.java | 2 +-
.../java/org/apache/cxf/wsdl11/SchemaUtil.java | 2 +-
.../org/apache/cxf/wsdl11/WSDLServiceBuilder.java | 4 ++--
.../apache/cxf/wsdl11/WSDLServiceBuilderTest.java | 2 +-
.../apache/cxf/sts/claims/LdapClaimsHandler.java | 2 +-
.../java/org/apache/cxf/sts/claims/LdapUtils.java | 6 +++---
.../apache/cxf/sts/claims/mapper/ClaimUtils.java | 2 +-
.../cxf/sts/operation/TokenIssueOperation.java | 2 +-
.../org/apache/cxf/sts/request/RequestParser.java | 2 +-
.../cxf/sts/token/canceller/SCTCanceller.java | 2 +-
.../cxf/sts/token/provider/SAMLTokenProvider.java | 6 +++---
.../cxf/sts/token/renewer/SAMLTokenRenewer.java | 2 +-
.../cxf/sts/common/CustomClaimsHandler.java | 2 +-
.../cxf/sts/common/CustomUserClaimsHandler.java | 2 +-
.../cxf/sts/common/RealmSupportClaimsHandler.java | 2 +-
.../systest/sts/custom/CustomClaimsHandler.java | 2 +-
.../sts/deployment/CustomClaimsHandler.java | 2 +-
.../sts/secure_conv/SCTSAMLTokenProvider.java | 2 +-
.../sts/deployment/CustomClaimsHandler.java | 2 +-
.../cxf/systest/sts/issueunit/IssueUnitTest.java | 14 +++++++-------
.../apache/cxf/systest/sts/rest/STSRESTTest.java | 12 ++++++------
.../cxf/systest/sts/itests/unit/STSUnitTest.java | 2 +-
.../apache/cxf/systest/jaxb/HashMapAdapter.java | 2 +-
.../systest/jaxrs/JAXRSClientServerBookTest.java | 2 +-
.../jaxrs/JAXRSClientServerStreamingTest.java | 2 +-
.../cxf/systest/ws/tokens/DoubleItBSTImpl.java | 4 ++--
.../org/apache/cxf/tools/common/ToolContext.java | 2 +-
.../cxf/tools/common/toolspec/ToolSpec.java | 6 +++---
.../common/toolspec/parser/CommandLineParser.java | 4 ++--
.../cxf/tools/common/toolspec/parser/Option.java | 8 ++++----
.../java/org/apache/cxf/tools/util/JAXBUtils.java | 8 ++++----
.../tools/corba/common/ProcessorEnvironment.java | 2 +-
.../cxf/tools/corba/common/idltypes/IdlRoot.java | 2 +-
.../idlpreprocessor/IdlPreprocessorReader.java | 2 +-
.../corba/processors/wsdl/WSDLParameter.java | 2 +-
.../corba/processors/wsdl/WSDLToCorbaBinding.java | 2 +-
.../processors/wsdl/WSDLToCorbaProcessor.java | 2 +-
.../validator/internal/ValidationResult.java | 6 +++---
.../cxf/tools/wadlto/jaxrs/SourceGenerator.java | 10 +++++-----
.../cxf/tools/wsdlto/core/PluginLoader.java | 2 +-
.../jaxws/generators/HandlerConfigGenerator.java | 2 +-
.../processor/internal/ParameterProcessor.java | 4 ++--
.../jaxws/validator/UniqueBodyValidator.java | 2 +-
.../tools/wsdlto/jaxws/JAXWSContainerTest.java | 2 +-
112 files changed, 176 insertions(+), 182 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java b/core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java
index c6dd731..0e7275a 100644
--- a/core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java
+++ b/core/src/main/java/org/apache/cxf/attachment/AttachmentUtil.java
@@ -350,13 +350,13 @@ public final class AttachmentUtil {
}
static String getHeaderValue(List<String> v) {
- if (v != null && v.size() > 0) {
+ if (v != null && !v.isEmpty()) {
return v.get(0);
}
return null;
}
static String getHeaderValue(List<String> v, String delim) {
- if (v != null && v.size() > 0) {
+ if (v != null && !v.isEmpty()) {
StringBuilder b = new StringBuilder();
for (String s : v) {
if (b.length() > 0) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/bus/extension/ExtensionManagerImpl.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/bus/extension/ExtensionManagerImpl.java b/core/src/main/java/org/apache/cxf/bus/extension/ExtensionManagerImpl.java
index ce71ac8..7a49518 100644
--- a/core/src/main/java/org/apache/cxf/bus/extension/ExtensionManagerImpl.java
+++ b/core/src/main/java/org/apache/cxf/bus/extension/ExtensionManagerImpl.java
@@ -33,11 +33,9 @@ import java.util.List;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.CopyOnWriteArrayList;
-import java.util.logging.Logger;
import org.apache.cxf.Bus;
import org.apache.cxf.common.injection.ResourceInjector;
-import org.apache.cxf.common.logging.LogUtils;
import org.apache.cxf.configuration.ConfiguredBeanLocator;
import org.apache.cxf.configuration.Configurer;
import org.apache.cxf.resource.ObjectTypeResolver;
@@ -46,9 +44,6 @@ import org.apache.cxf.resource.ResourceResolver;
import org.apache.cxf.resource.SinglePropertyResolver;
public class ExtensionManagerImpl implements ExtensionManager, ConfiguredBeanLocator {
- private static final Logger LOG = LogUtils.getL7dLogger(ExtensionManagerImpl.class);
-
-
public static final String EXTENSIONMANAGER_PROPERTY_NAME = "extensionManager";
public static final String ACTIVATION_NAMESPACES_PROPERTY_NAME = "activationNamespaces";
public static final String ACTIVATION_NAMESPACES_SETTER_METHOD_NAME = "setActivationNamespaces";
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java b/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
index 8c827b0..83364b8 100644
--- a/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
+++ b/core/src/main/java/org/apache/cxf/common/jaxb/JAXBUtils.java
@@ -92,6 +92,8 @@ import org.apache.cxf.common.xmlschema.SchemaCollection;
import org.apache.cxf.helpers.JavaUtils;
public final class JAXBUtils {
+ public static final String JAXB_URI = "http://java.sun.com/xml/ns/jaxb";
+
private static final Logger LOG = LogUtils.getL7dLogger(JAXBUtils.class);
public enum IdentifierType {
@@ -103,9 +105,6 @@ public final class JAXBUtils {
CONSTANT
};
- public static final String JAXB_URI = "http://java.sun.com/xml/ns/jaxb";
-
-
private static final char[] XML_NAME_PUNCTUATION_CHARS = new char[] {
/* hyphen */ '\u002D',
/* period */ '\u002E',
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/databinding/stax/StaxDataBindingInterceptor.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/databinding/stax/StaxDataBindingInterceptor.java b/core/src/main/java/org/apache/cxf/databinding/stax/StaxDataBindingInterceptor.java
index 580afac..7c8ab6d 100644
--- a/core/src/main/java/org/apache/cxf/databinding/stax/StaxDataBindingInterceptor.java
+++ b/core/src/main/java/org/apache/cxf/databinding/stax/StaxDataBindingInterceptor.java
@@ -77,7 +77,7 @@ public class StaxDataBindingInterceptor extends AbstractInDatabindingInterceptor
}
- if (parameters.size() > 0) {
+ if (!parameters.isEmpty()) {
message.setContent(List.class, parameters);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/interceptor/ClientFaultConverter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/interceptor/ClientFaultConverter.java b/core/src/main/java/org/apache/cxf/interceptor/ClientFaultConverter.java
index 4834340..5d2e7db 100644
--- a/core/src/main/java/org/apache/cxf/interceptor/ClientFaultConverter.java
+++ b/core/src/main/java/org/apache/cxf/interceptor/ClientFaultConverter.java
@@ -255,7 +255,7 @@ public class ClientFaultConverter extends AbstractInDatabindingInterceptor {
}
stackTraceList.add(parseStackTrackLine(oneLine));
}
- if (stackTraceList.size() > 0 || cause != null) {
+ if (!stackTraceList.isEmpty() || cause != null) {
Exception e = msg.getContent(Exception.class);
if (!stackTraceList.isEmpty()) {
StackTraceElement[] stackTraceElement = new StackTraceElement[stackTraceList.size()];
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/message/MessageImpl.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/message/MessageImpl.java b/core/src/main/java/org/apache/cxf/message/MessageImpl.java
index 1b3691b..0c2453d 100644
--- a/core/src/main/java/org/apache/cxf/message/MessageImpl.java
+++ b/core/src/main/java/org/apache/cxf/message/MessageImpl.java
@@ -184,7 +184,7 @@ public class MessageImpl extends StringMapImpl implements Message {
private static final long serialVersionUID = 7067290677790419348L;
public void putAll(Map<? extends String, ? extends Object> m) {
- if (m != null && m.size() > 0) {
+ if (m != null && !m.isEmpty()) {
super.putAll(m);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/service/model/OperationInfo.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/service/model/OperationInfo.java b/core/src/main/java/org/apache/cxf/service/model/OperationInfo.java
index b4a64ac..69aa5f3 100644
--- a/core/src/main/java/org/apache/cxf/service/model/OperationInfo.java
+++ b/core/src/main/java/org/apache/cxf/service/model/OperationInfo.java
@@ -189,7 +189,7 @@ public class OperationInfo extends AbstractPropertiesHolder implements NamedItem
}
public boolean hasFaults() {
- return faults != null && faults.size() > 0;
+ return faults != null && !faults.isEmpty();
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/staxutils/W3CDOMStreamWriter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/staxutils/W3CDOMStreamWriter.java b/core/src/main/java/org/apache/cxf/staxutils/W3CDOMStreamWriter.java
index ae40f6e..f44a9fb 100644
--- a/core/src/main/java/org/apache/cxf/staxutils/W3CDOMStreamWriter.java
+++ b/core/src/main/java/org/apache/cxf/staxutils/W3CDOMStreamWriter.java
@@ -183,7 +183,7 @@ public class W3CDOMStreamWriter implements XMLStreamWriter {
}
public void writeEndElement() throws XMLStreamException {
- if (stack.size() > 0) {
+ if (!stack.isEmpty()) {
currentNode = stack.pop();
} else {
currentNode = null;
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java b/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
index 56da591..a7ade15 100644
--- a/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
+++ b/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
@@ -399,7 +399,7 @@ public class OutTransformWriter extends DelegatingXMLStreamWriter {
private boolean matchesDropped(boolean shallow) {
return (dropDepth > 0 && dropDepth <= currentDepth)
- || (shallow && (elementsStack.size() > 0 && dropElements.contains(elementsStack.get(0))));
+ || (shallow && (!elementsStack.isEmpty() && dropElements.contains(elementsStack.get(0))));
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/core/src/main/java/org/apache/cxf/validation/AbstractBeanValidationInterceptor.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/validation/AbstractBeanValidationInterceptor.java b/core/src/main/java/org/apache/cxf/validation/AbstractBeanValidationInterceptor.java
index 0a04df1..80fac76 100644
--- a/core/src/main/java/org/apache/cxf/validation/AbstractBeanValidationInterceptor.java
+++ b/core/src/main/java/org/apache/cxf/validation/AbstractBeanValidationInterceptor.java
@@ -50,7 +50,7 @@ public abstract class AbstractBeanValidationInterceptor extends AbstractValidati
@Override
protected void handleValidation(final Message message, final Object resourceInstance,
final Method method, final List<Object> arguments) {
- if (arguments.size() > 0) {
+ if (!arguments.isEmpty()) {
BeanValidationProvider provider = getProvider(message);
provider.validateParameters(resourceInstance, method, unwrapArgs(arguments).toArray());
message.getExchange().put(BeanValidationProvider.class, provider);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectReader.java
----------------------------------------------------------------------
diff --git a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectReader.java b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectReader.java
index 328de06..2c98528 100644
--- a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectReader.java
+++ b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectReader.java
@@ -364,7 +364,7 @@ public class CorbaObjectReader {
Union unionType = (Union)unionHandler.getType();
List<Unionbranch> branches = unionType.getUnionbranch();
CorbaObjectHandler discriminator = unionHandler.getDiscriminator();
- if (branches.size() > 0) {
+ if (!branches.isEmpty()) {
String discLabel = null;
if (discriminator.getTypeCodeKind().value() == TCKind._tk_enum) {
CorbaEnumHandler disc = (CorbaEnumHandler) discriminator;
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectWriter.java
----------------------------------------------------------------------
diff --git a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectWriter.java b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectWriter.java
index a6af129..d4de7c4 100644
--- a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectWriter.java
+++ b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/runtime/CorbaObjectWriter.java
@@ -311,7 +311,7 @@ public class CorbaObjectWriter {
public void writeUnion(CorbaObjectHandler obj) throws CorbaBindingException {
Union unionType = (Union) obj.getType();
List<Unionbranch> branches = unionType.getUnionbranch();
- if (branches.size() > 0) {
+ if (!branches.isEmpty()) {
CorbaObjectHandler discriminator = ((CorbaUnionHandler)obj).getDiscriminator();
this.write(discriminator);
CorbaObjectHandler unionValue = ((CorbaUnionHandler)obj).getValue();
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
----------------------------------------------------------------------
diff --git a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
index 26db8dc..9fec946 100644
--- a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
+++ b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
@@ -243,7 +243,7 @@ public class SoapBindingFactory extends AbstractWSDLBindingFactory {
if (b.getInput() != null) {
List<String> bodyParts = null;
List<SoapHeaderInfo> headerInfos = b.getInput().getExtensors(SoapHeaderInfo.class);
- if (headerInfos != null && headerInfos.size() > 0) {
+ if (headerInfos != null && !headerInfos.isEmpty()) {
bodyParts = new ArrayList<>();
for (MessagePartInfo part : b.getInput().getMessageParts()) {
bodyParts.add(part.getName().getLocalPart());
@@ -279,7 +279,7 @@ public class SoapBindingFactory extends AbstractWSDLBindingFactory {
if (b.getOutput() != null) {
List<String> bodyParts = null;
List<SoapHeaderInfo> headerInfos = b.getOutput().getExtensors(SoapHeaderInfo.class);
- if (headerInfos != null && headerInfos.size() > 0) {
+ if (headerInfos != null && !headerInfos.isEmpty()) {
bodyParts = new ArrayList<>();
for (MessagePartInfo part : b.getOutput().getMessageParts()) {
bodyParts.add(part.getName().getLocalPart());
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapFault.java
----------------------------------------------------------------------
diff --git a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapFault.java b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapFault.java
index 6e74c22..41470e2 100644
--- a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapFault.java
+++ b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapFault.java
@@ -104,7 +104,7 @@ public class SoapFault extends Fault {
}
private QName getRootSubCode() {
- return subCodes != null && subCodes.size() > 0 ? subCodes.get(0) : null;
+ return subCodes != null && !subCodes.isEmpty() ? subCodes.get(0) : null;
}
private void setRootSubCode(QName subCode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionInInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionInInterceptor.java b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionInInterceptor.java
index 94e899b..ef143d4 100644
--- a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionInInterceptor.java
+++ b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapActionInInterceptor.java
@@ -65,7 +65,7 @@ public class SoapActionInInterceptor extends AbstractSoapInterceptor {
= CastUtils.cast((Map<?, ?>)message.get(Message.PROTOCOL_HEADERS));
if (headers != null) {
List<String> sa = headers.get(SoapBindingConstants.SOAP_ACTION);
- if (sa != null && sa.size() > 0) {
+ if (sa != null && !sa.isEmpty()) {
String action = sa.get(0);
if (action.startsWith("\"")) {
action = action.substring(1, action.length() - 1);
@@ -86,7 +86,7 @@ public class SoapActionInInterceptor extends AbstractSoapInterceptor {
// but skip searching if the start-info property is set
List<String> cts = CastUtils.cast((List<?>)(((Map<?, ?>)
message.get(AttachmentDeserializer.ATTACHMENT_PART_HEADERS)).get(Message.CONTENT_TYPE)));
- if (cts != null && cts.size() > 0) {
+ if (cts != null && !cts.isEmpty()) {
ct = cts.get(0);
start = ct.indexOf("action=");
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderInterceptor.java b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderInterceptor.java
index 6cba304..3b63888 100644
--- a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderInterceptor.java
+++ b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/interceptor/SoapHeaderInterceptor.java
@@ -160,7 +160,7 @@ public class SoapHeaderInterceptor extends AbstractInDatabindingInterceptor {
parameters.put(mpi, object);
}
}
- if (parameters.size() > 0) {
+ if (!parameters.isEmpty()) {
message.setContent(List.class, parameters);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/jms/interceptor/SoapJMSInInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/jms/interceptor/SoapJMSInInterceptor.java b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/jms/interceptor/SoapJMSInInterceptor.java
index 840e6ab..8aca871 100644
--- a/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/jms/interceptor/SoapJMSInInterceptor.java
+++ b/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/jms/interceptor/SoapJMSInInterceptor.java
@@ -62,7 +62,7 @@ public class SoapJMSInInterceptor extends AbstractSoapInterceptor {
private void checkContentEncoding(SoapMessage message, Map<String, List<String>> headers) {
List<String> contentEncodingList = headers.get(SoapJMSConstants.CONTENTENCODING_FIELD);
JMSFault jmsFault = null;
- if (contentEncodingList != null && contentEncodingList.size() > 0) {
+ if (contentEncodingList != null && !contentEncodingList.isEmpty()) {
String contentEncoding = contentEncodingList.get(0);
if (!"gzip".equals(contentEncoding)) {
jmsFault = JMSFaultFactory.createContentEncodingNotSupported(contentEncoding);
@@ -83,7 +83,7 @@ public class SoapJMSInInterceptor extends AbstractSoapInterceptor {
*/
private void checkJMSMessageFormat(SoapMessage message, Map<String, List<String>> headers) {
List<String> mt = headers.get(SoapJMSConstants.JMS_MESSAGE_TYPE);
- if (mt != null && mt.size() > 0) {
+ if (mt != null && !mt.isEmpty()) {
String messageType = mt.get(0);
if (!"text".equals(messageType) && !"byte".equals(messageType)) {
JMSFault jmsFault = JMSFaultFactory.createUnsupportedJMSMessageFormatFault(messageType);
@@ -106,13 +106,13 @@ public class SoapJMSInInterceptor extends AbstractSoapInterceptor {
String contentTypeAction = null;
List<String> ct = headers.get(SoapJMSConstants.CONTENTTYPE_FIELD);
List<String> sa = headers.get(SoapJMSConstants.SOAPACTION_FIELD);
- if (sa != null && sa.size() > 0) {
+ if (sa != null && !sa.isEmpty()) {
soapAction = sa.get(0);
if (soapAction != null && soapAction.startsWith("\"")) {
soapAction = soapAction.substring(1, soapAction.lastIndexOf("\""));
}
}
- if (ct != null && ct.size() > 0) {
+ if (ct != null && !ct.isEmpty()) {
contentType = ct.get(0);
}
if (contentType != null && contentType.indexOf("action=") != -1) {
@@ -148,15 +148,15 @@ public class SoapJMSInInterceptor extends AbstractSoapInterceptor {
private void checkRequestURI(SoapMessage message, Map<String, List<String>> headers) {
List<String> ru = headers.get(SoapJMSConstants.REQUESTURI_FIELD);
JMSFault jmsFault = null;
- if (ru != null && ru.size() > 0) {
+ if (ru != null && !ru.isEmpty()) {
String requestURI = ru.get(0);
List<String> mr = headers.get(SoapJMSConstants.MALFORMED_REQUESTURI);
- if (mr != null && mr.size() > 0 && mr.get(0).equals("true")) {
+ if (mr != null && !mr.isEmpty() && mr.get(0).equals("true")) {
jmsFault = JMSFaultFactory.createMalformedRequestURIFault(requestURI);
}
List<String> trn = headers.get(SoapJMSConstants.TARGET_SERVICE_IN_REQUESTURI);
- if (trn != null && trn.size() > 0 && trn.get(0).equals("true")) {
+ if (trn != null && !trn.isEmpty() && trn.get(0).equals("true")) {
jmsFault = JMSFaultFactory.createTargetServiceNotAllowedInRequestURIFault();
}
} else {
@@ -177,7 +177,7 @@ public class SoapJMSInInterceptor extends AbstractSoapInterceptor {
private void checkContentType(SoapMessage message, Map<String, List<String>> headers) {
List<String> ct = headers.get(SoapJMSConstants.CONTENTTYPE_FIELD);
JMSFault jmsFault = null;
- if (ct != null && ct.size() > 0) {
+ if (ct != null && !ct.isEmpty()) {
String contentType = ct.get(0);
if (!contentType.startsWith("text/xml")
&& !contentType.startsWith("application/soap+xml")
@@ -203,7 +203,7 @@ public class SoapJMSInInterceptor extends AbstractSoapInterceptor {
*/
private void checkBindingVersion(SoapMessage message, Map<String, List<String>> headers) {
List<String> bv = headers.get(SoapJMSConstants.BINDINGVERSION_FIELD);
- if (bv != null && bv.size() > 0) {
+ if (bv != null && !bv.isEmpty()) {
String bindingVersion = bv.get(0);
if (!"1.0".equals(bindingVersion)) {
JMSFault jmsFault = JMSFaultFactory
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
index 582deb3..f27daf2 100644
--- a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
+++ b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/databinding/AegisDatabinding.java
@@ -223,7 +223,7 @@ public class AegisDatabinding extends AbstractDataBinding {
aegisContext.setTypeCreationOptions(configuration);
}
- if (implMap.size() > 0) {
+ if (!implMap.isEmpty()) {
aegisContext.setBeanImplementationMap(implMap);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/XMLTypeCreator.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/XMLTypeCreator.java b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/XMLTypeCreator.java
index f3ded00..db435ac 100644
--- a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/XMLTypeCreator.java
+++ b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/XMLTypeCreator.java
@@ -350,7 +350,7 @@ public class XMLTypeCreator extends AbstractTypeCreator {
List<Element> mappings = findMappings(info.getType());
Class<?> relatedClass = TypeUtil.getTypeRelatedClass(info.getType());
- if (mapping != null || mappings.size() > 0) {
+ if (mapping != null || !mappings.isEmpty()) {
String typeNameAtt = null;
if (mapping != null) {
typeNameAtt = DOMUtils.getAttributeValueEmptyNull(mapping, "name");
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamReader.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamReader.java b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamReader.java
index 16e90fc..99dd0de 100644
--- a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamReader.java
+++ b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/util/stax/JDOMStreamReader.java
@@ -174,7 +174,7 @@ public class JDOMStreamReader extends AbstractDOMStreamReader<Element, Integer>
@Override
protected void endElement() {
- if (namespaceStack.size() > 0) {
+ if (!namespaceStack.isEmpty()) {
prefix2decNs = namespaceStack.pop();
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/features/clustering/src/main/java/org/apache/cxf/clustering/AbstractStaticFailoverStrategy.java
----------------------------------------------------------------------
diff --git a/rt/features/clustering/src/main/java/org/apache/cxf/clustering/AbstractStaticFailoverStrategy.java b/rt/features/clustering/src/main/java/org/apache/cxf/clustering/AbstractStaticFailoverStrategy.java
index ebef316..599f5e8 100644
--- a/rt/features/clustering/src/main/java/org/apache/cxf/clustering/AbstractStaticFailoverStrategy.java
+++ b/rt/features/clustering/src/main/java/org/apache/cxf/clustering/AbstractStaticFailoverStrategy.java
@@ -76,7 +76,7 @@ public abstract class AbstractStaticFailoverStrategy implements FailoverStrategy
*/
public String selectAlternateAddress(List<String> alternates) {
String selected = null;
- if (alternates != null && alternates.size() > 0) {
+ if (alternates != null && !alternates.isEmpty()) {
selected = getNextAlternate(alternates);
Level level = getLogLevel();
if (LOG.isLoggable(level)) {
@@ -108,7 +108,7 @@ public abstract class AbstractStaticFailoverStrategy implements FailoverStrategy
*/
public Endpoint selectAlternateEndpoint(List<Endpoint> alternates) {
Endpoint selected = null;
- if (alternates != null && alternates.size() > 0) {
+ if (alternates != null && !alternates.isEmpty()) {
selected = getNextAlternate(alternates);
Level level = getLogLevel();
if (LOG.isLoggable(level)) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/AbstractJAXRSFactoryBean.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/AbstractJAXRSFactoryBean.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/AbstractJAXRSFactoryBean.java
index 122cec5..9481101 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/AbstractJAXRSFactoryBean.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/AbstractJAXRSFactoryBean.java
@@ -345,7 +345,7 @@ public class AbstractJAXRSFactoryBean extends AbstractEndpointFactory {
protected void setDataBindingProvider(ProviderFactory factory, Service s) {
List<ClassResourceInfo> cris = serviceFactory.getRealClassResourceInfo();
- if (getDataBinding() == null && cris.size() > 0) {
+ if (getDataBinding() == null && !cris.isEmpty()) {
org.apache.cxf.annotations.DataBinding ann =
cris.get(0).getServiceClass().getAnnotation(org.apache.cxf.annotations.DataBinding.class);
if (ann != null) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/ProtocolHeadersImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/ProtocolHeadersImpl.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/ProtocolHeadersImpl.java
index 697341a..9539f01 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/ProtocolHeadersImpl.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/ProtocolHeadersImpl.java
@@ -42,7 +42,7 @@ public class ProtocolHeadersImpl implements ProtocolHeaders {
public String getRequestHeaderValue(String name) {
List<String> values = getRequestHeader(name);
- return values.size() > 0 ? values.get(0) : null;
+ return !values.isEmpty() ? values.get(0) : null;
}
public MultivaluedMap<String, String> getRequestHeaders() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/multipart/MultipartBody.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/multipart/MultipartBody.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/multipart/MultipartBody.java
index bafc333..b3e2ca3 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/multipart/MultipartBody.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/multipart/MultipartBody.java
@@ -79,7 +79,7 @@ public class MultipartBody {
}
public Attachment getRootAttachment() {
- return atts.size() > 0 ? atts.get(0) : null;
+ return !atts.isEmpty() ? atts.get(0) : null;
}
public Attachment getAttachment(String contentId) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/RequestImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/RequestImpl.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/RequestImpl.java
index 221c25a..aa64910 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/RequestImpl.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/RequestImpl.java
@@ -107,7 +107,7 @@ public class RequestImpl implements Request {
matchingVars.add(var);
}
}
- if (matchingVars.size() > 0) {
+ if (!matchingVars.isEmpty()) {
addVaryHeader(varyValues);
Collections.sort(matchingVars, new VariantComparator());
return matchingVars.get(0);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/VariantListBuilderImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/VariantListBuilderImpl.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/VariantListBuilderImpl.java
index ea04001..ce1cc2d 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/VariantListBuilderImpl.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/VariantListBuilderImpl.java
@@ -78,11 +78,11 @@ public class VariantListBuilderImpl extends VariantListBuilder {
}
private void addVariants() {
- if (mediaTypes.size() > 0) {
+ if (!mediaTypes.isEmpty()) {
handleMediaTypes();
- } else if (languages.size() > 0) {
+ } else if (!languages.isEmpty()) {
handleLanguages(null);
- } else if (encodings.size() > 0) {
+ } else if (!encodings.isEmpty()) {
for (String enc : encodings) {
variants.add(new Variant(null, (Locale)null, enc));
}
@@ -91,9 +91,9 @@ public class VariantListBuilderImpl extends VariantListBuilder {
private void handleMediaTypes() {
for (MediaType type : mediaTypes) {
- if (languages.size() > 0) {
+ if (!languages.isEmpty()) {
handleLanguages(type);
- } else if (encodings.size() > 0) {
+ } else if (!encodings.isEmpty()) {
for (String enc : encodings) {
variants.add(new Variant(type, (Locale)null, enc));
}
@@ -105,7 +105,7 @@ public class VariantListBuilderImpl extends VariantListBuilder {
private void handleLanguages(MediaType type) {
for (Locale lang : languages) {
- if (encodings.size() > 0) {
+ if (!encodings.isEmpty()) {
for (String enc : encodings) {
variants.add(new Variant(type, lang, enc));
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/AbstractJAXBProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/AbstractJAXBProvider.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/AbstractJAXBProvider.java
index d8c35e9..01aa8c9 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/AbstractJAXBProvider.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/AbstractJAXBProvider.java
@@ -872,7 +872,7 @@ public abstract class AbstractJAXBProvider<T> extends AbstractConfigurableProvid
XmlJavaTypeAdapter adapter) throws JAXBException {
List<?> theList = getList();
boolean adapterChecked = false;
- if (theList.size() > 0) {
+ if (!theList.isEmpty()) {
Object first = theList.get(0);
if (first instanceof Element) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
index 190574c..c3ca9fd 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
@@ -386,7 +386,7 @@ public class RequestDispatcherProvider extends AbstractConfigurableProvider
}
List<PathSegment> segments = ui.getPathSegments();
- if (segments.size() > 0) {
+ if (!segments.isEmpty()) {
doSetRequestParameters(request, segments.get(segments.size() - 1).getMatrixParameters());
}
doSetRequestParameters(request, ui.getQueryParameters());
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
index fe57120..d65e451 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/ServerProviderFactory.java
@@ -357,7 +357,7 @@ public final class ServerProviderFactory extends ProviderFactory {
}
public void applyDynamicFeatures(List<ClassResourceInfo> list) {
- if (dynamicFeatures.size() > 0) {
+ if (!dynamicFeatures.isEmpty()) {
for (ClassResourceInfo cri : list) {
doApplyDynamicFeatures(cri);
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/XSLTJaxbProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/XSLTJaxbProvider.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/XSLTJaxbProvider.java
index 29a1855..90c6a1b 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/XSLTJaxbProvider.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/XSLTJaxbProvider.java
@@ -464,7 +464,7 @@ public class XSLTJaxbProvider<T> extends JAXBElementProvider<T> {
}
List<PathSegment> segments = ui.getPathSegments();
- if (segments.size() > 0) {
+ if (!segments.isEmpty()) {
setTransformParameters(templ, segments.get(segments.size() - 1).getMatrixParameters());
}
setTransformParameters(templ, ui.getQueryParameters());
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/security/JAASAuthenticationFilter.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/security/JAASAuthenticationFilter.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/security/JAASAuthenticationFilter.java
index b764e25..fbdbac4 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/security/JAASAuthenticationFilter.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/security/JAASAuthenticationFilter.java
@@ -145,7 +145,7 @@ public class JAASAuthenticationFilter implements ContainerRequestFilter {
StringBuilder sb = new StringBuilder();
List<String> authHeader = headers.getRequestHeader(HttpHeaders.AUTHORIZATION);
- if (authHeader != null && authHeader.size() > 0) {
+ if (authHeader != null && !authHeader.isEmpty()) {
// should HttpHeadersImpl do it ?
String[] authValues = StringUtils.split(authHeader.get(0), " ");
if (authValues.length > 0) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
index bf1f40f..1ad1fe9 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
@@ -211,7 +211,7 @@ public class CXFNonSpringJaxrsServlet extends CXFNonSpringServlet {
list.add(theLoc);
}
}
- if (list.size() > 0) {
+ if (!list.isEmpty()) {
bean.setSchemaLocations(list);
}
}
@@ -250,7 +250,7 @@ public class CXFNonSpringJaxrsServlet extends CXFNonSpringServlet {
}
}
}
- if (list.size() > 0) {
+ if (!list.isEmpty()) {
if (OUT_INTERCEPTORS_PARAM.equals(paramName)) {
bean.setOutInterceptors(list);
} else if (OUT_FAULT_INTERCEPTORS_PARAM.equals(paramName)) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/InjectionUtils.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/InjectionUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/InjectionUtils.java
index c52a984..a990c22 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/InjectionUtils.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/InjectionUtils.java
@@ -610,7 +610,7 @@ public final class InjectionUtils {
value.put(memberKey, entry.getValue());
}
- if (parsedValues.size() > 0) {
+ if (!parsedValues.isEmpty()) {
for (Map.Entry<String, MultivaluedMap<String, String>> entry : parsedValues.entrySet()) {
String memberKey = entry.getKey();
@@ -999,7 +999,7 @@ public final class InjectionUtils {
paramValuesMap, false, decoded, pathParam, message);
} else {
String result = null;
- if (paramValues.size() > 0) {
+ if (!paramValues.isEmpty()) {
boolean isLast = pathParam == ParameterType.PATH ? true : false;
result = isLast ? paramValues.get(paramValues.size() - 1)
: paramValues.get(0);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
index 415c8ca..f3643f9 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
@@ -209,7 +209,7 @@ public final class JAXRSUtils {
types = ((AbstractConfigurableProvider)provider).getProduceMediaTypes();
}
if (types != null) {
- values = types.size() > 0 ? types.toArray(new String[types.size()])
+ values = !types.isEmpty() ? types.toArray(new String[types.size()])
: new String[]{"*/*"};
}
}
@@ -906,7 +906,7 @@ public final class JAXRSUtils {
boolean decode) {
List<PathSegment> segments = JAXRSUtils.getPathSegments(
(String)m.get(Message.REQUEST_URI), decode);
- if (segments.size() > 0) {
+ if (!segments.isEmpty()) {
MultivaluedMap<String, String> params = new MetadataMap<String, String>();
for (PathSegment ps : segments) {
MultivaluedMap<String, String> matrix = ps.getMatrixParameters();
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
index 7f8541b..ce71701 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
@@ -147,7 +147,7 @@ public final class AttachmentUtils {
}
}
- return infos.size() > 0 ? infos.get(0) : null;
+ return !infos.isEmpty() ? infos.get(0) : null;
}
public static List<Attachment> getMatchingAttachments(Multipart id,
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/EndpointImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/EndpointImpl.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/EndpointImpl.java
index a1be6e7..7752e6d 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/EndpointImpl.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/EndpointImpl.java
@@ -451,7 +451,7 @@ public class EndpointImpl extends javax.xml.ws.Endpoint
if (executor != null) {
serverFactory.getServiceFactory().setExecutor(executor);
}
- if (handlers.size() > 0) {
+ if (!handlers.isEmpty()) {
serverFactory.addHandlers(handlers);
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
index b83e511..502a0f9 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/spi/ProviderImpl.java
@@ -278,7 +278,7 @@ public class ProviderImpl extends javax.xml.ws.spi.Provider {
if (wsdlDocumentLocation != null
|| interfaceName != null
|| serviceName != null
- || (metadata != null && metadata.size() > 0)) {
+ || (metadata != null && !metadata.isEmpty())) {
writer.writeStartElement(JAXWSAConstants.WSA_PREFIX, JAXWSAConstants.WSA_METADATA_NAME,
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsImplementorInfo.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsImplementorInfo.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsImplementorInfo.java
index a2fa342..075d683 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsImplementorInfo.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsImplementorInfo.java
@@ -103,7 +103,7 @@ public class JaxWsImplementorInfo {
String namespace = null;
// serviceName cannot be specified on SEI so check impl class only
- if (wsAnnotations.size() > 0) {
+ if (!wsAnnotations.isEmpty()) {
int offset = 1;
if (seiClass == null) {
offset = 0;
@@ -148,7 +148,7 @@ public class JaxWsImplementorInfo {
String name = null;
// portName cannot be specified on SEI so check impl class only
- if (wsAnnotations.size() > 0) {
+ if (!wsAnnotations.isEmpty()) {
int offset = 1;
if (seiClass == null) {
offset = 0;
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsServiceFactoryBean.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsServiceFactoryBean.java b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsServiceFactoryBean.java
index 5d09217..7923ff5 100644
--- a/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsServiceFactoryBean.java
+++ b/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/support/JaxWsServiceFactoryBean.java
@@ -184,7 +184,7 @@ public class JaxWsServiceFactoryBean extends ReflectionServiceFactoryBean {
features.add(new RespectBindingFeature(respectBinding.enabled()));
}
- if (features.size() > 0) {
+ if (!features.isEmpty()) {
wsFeatures = features;
if (setWsFeatures != null) {
wsFeatures.addAll(setWsFeatures);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/frontend/simple/src/main/java/org/apache/cxf/endpoint/dynamic/DynamicClientFactory.java
----------------------------------------------------------------------
diff --git a/rt/frontend/simple/src/main/java/org/apache/cxf/endpoint/dynamic/DynamicClientFactory.java b/rt/frontend/simple/src/main/java/org/apache/cxf/endpoint/dynamic/DynamicClientFactory.java
index bcae789..bf503fe 100644
--- a/rt/frontend/simple/src/main/java/org/apache/cxf/endpoint/dynamic/DynamicClientFactory.java
+++ b/rt/frontend/simple/src/main/java/org/apache/cxf/endpoint/dynamic/DynamicClientFactory.java
@@ -367,7 +367,7 @@ public class DynamicClientFactory {
}
List<File> srcFiles = FileUtils.getFilesRecurse(src, ".+\\.java$");
- if (srcFiles.size() > 0 && !compileJavaSrc(classPath.toString(), srcFiles, classes.toString())) {
+ if (!srcFiles.isEmpty() && !compileJavaSrc(classPath.toString(), srcFiles, classes.toString())) {
LOG.log(Level.SEVERE, new Message("COULD_NOT_COMPILE_SRC", LOG, wsdlUrl).toString());
}
FileUtils.removeDir(src);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/JsSimpleDomNode.java
----------------------------------------------------------------------
diff --git a/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/JsSimpleDomNode.java b/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/JsSimpleDomNode.java
index 4c3ab48..ca34be5 100644
--- a/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/JsSimpleDomNode.java
+++ b/rt/javascript/javascript-tests/src/test/java/org/apache/cxf/javascript/JsSimpleDomNode.java
@@ -83,7 +83,7 @@ public class JsSimpleDomNode extends ScriptableObject {
public Object jsGet_firstChild() {
establishChildren();
- if (children.size() > 0)
+ if (!children.isEmpty())
return children.get(0);
else
return null;
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
----------------------------------------------------------------------
diff --git a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
index 864a3fe..e34de1a 100644
--- a/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
+++ b/rt/rs/description/src/test/java/org/apache/cxf/jaxrs/model/wadl/WadlGeneratorTest.java
@@ -397,7 +397,7 @@ public class WadlGeneratorTest extends Assert {
}
private void checkGrammarsWithLinks(Element appElement, List<String> links) {
- assertTrue(links.size() > 0);
+ assertTrue(!links.isEmpty());
List<Element> grammarEls = DOMUtils.getChildrenWithName(appElement, WadlGenerator.WADL_NS,
"grammars");
assertEquals(1, grammarEls.size());
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/rs/extensions/rx/src/main/java/org/apache/cxf/jaxrs/rx/server/StreamingAsyncSubscriber.java
----------------------------------------------------------------------
diff --git a/rt/rs/extensions/rx/src/main/java/org/apache/cxf/jaxrs/rx/server/StreamingAsyncSubscriber.java b/rt/rs/extensions/rx/src/main/java/org/apache/cxf/jaxrs/rx/server/StreamingAsyncSubscriber.java
index cd1b91f..1571973 100644
--- a/rt/rs/extensions/rx/src/main/java/org/apache/cxf/jaxrs/rx/server/StreamingAsyncSubscriber.java
+++ b/rt/rs/extensions/rx/src/main/java/org/apache/cxf/jaxrs/rx/server/StreamingAsyncSubscriber.java
@@ -88,7 +88,7 @@ public class StreamingAsyncSubscriber<T> extends AbstractAsyncSubscriber<T> {
if (openTag != null) {
writer.getEntityStream().write(StringUtils.toBytesUTF8(openTag));
}
- while (!completed || queue.size() > 0) {
+ while (!completed || !queue.isEmpty()) {
try {
T bean = queue.poll(pollTimeout, TimeUnit.MILLISECONDS);
if (bean != null) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SearchUtils.java
----------------------------------------------------------------------
diff --git a/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SearchUtils.java b/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SearchUtils.java
index 2f7082c..52dc444 100644
--- a/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SearchUtils.java
+++ b/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/SearchUtils.java
@@ -145,7 +145,7 @@ public final class SearchUtils {
String tableAlias,
List<String> columns) {
sb.append("SELECT ");
- if (columns != null && columns.size() > 0) {
+ if (columns != null && !columns.isEmpty()) {
for (int i = 0; i < columns.size(); i++) {
sb.append(columns.get(i));
if (i + 1 < columns.size()) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java b/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
index b7ab474..df0da4f 100644
--- a/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
+++ b/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/filters/AbstractAuthFilter.java
@@ -187,7 +187,7 @@ public class AbstractAuthFilter {
}
}
- if (permissions.size() > 0 && matchingPermissions.isEmpty()) {
+ if (!permissions.isEmpty() && matchingPermissions.isEmpty()) {
String message = "Client has no valid permissions";
LOG.warning(message);
throw new OAuthProblemException(message);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java
----------------------------------------------------------------------
diff --git a/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java b/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java
index f9dbaed..e7def27 100644
--- a/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java
+++ b/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java
@@ -435,7 +435,7 @@ public class JettyHTTPServerEngine implements ServerEngine {
* one to start, or it is now one containing only the single handler
* that was there to begin with.
*/
- if (handlers != null && handlers.size() > 0) {
+ if (handlers != null && !handlers.isEmpty()) {
for (Handler h : handlers) {
// Filtering out the jetty default handler
// which should not be added at this point.
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineFactoryBeanDefinitionParser.java
----------------------------------------------------------------------
diff --git a/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineFactoryBeanDefinitionParser.java b/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineFactoryBeanDefinitionParser.java
index 8b6b8f8..2e9e1c0 100644
--- a/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineFactoryBeanDefinitionParser.java
+++ b/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineFactoryBeanDefinitionParser.java
@@ -96,7 +96,7 @@ public class JettyHTTPServerEngineFactoryBeanDefinitionParser
// parser the engine list
List<Object> list =
getRequiredElementsList(element, ctx, new QName(HTTP_JETTY_NS, "engine"), bean);
- if (list.size() > 0) {
+ if (!list.isEmpty()) {
bean.addPropertyValue("enginesList", list);
}
} catch (Exception e) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/http-netty/netty-server/src/main/java/org/apache/cxf/transport/http/netty/server/spring/NettyHttpServerEngineFactoryBeanDefinitionParser.java
----------------------------------------------------------------------
diff --git a/rt/transports/http-netty/netty-server/src/main/java/org/apache/cxf/transport/http/netty/server/spring/NettyHttpServerEngineFactoryBeanDefinitionParser.java b/rt/transports/http-netty/netty-server/src/main/java/org/apache/cxf/transport/http/netty/server/spring/NettyHttpServerEngineFactoryBeanDefinitionParser.java
index c809299..210b673 100644
--- a/rt/transports/http-netty/netty-server/src/main/java/org/apache/cxf/transport/http/netty/server/spring/NettyHttpServerEngineFactoryBeanDefinitionParser.java
+++ b/rt/transports/http-netty/netty-server/src/main/java/org/apache/cxf/transport/http/netty/server/spring/NettyHttpServerEngineFactoryBeanDefinitionParser.java
@@ -96,7 +96,7 @@ public class NettyHttpServerEngineFactoryBeanDefinitionParser
// parser the engine list
List<Object> list =
getRequiredElementsList(element, ctx, new QName(HTTP_NETTY_SERVER_NS, "engine"), bean);
- if (list.size() > 0) {
+ if (!list.isEmpty()) {
bean.addPropertyValue("enginesList", list);
}
} catch (Exception e) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/http-undertow/src/main/java/org/apache/cxf/transport/http_undertow/spring/UndertowHTTPServerEngineFactoryBeanDefinitionParser.java
----------------------------------------------------------------------
diff --git a/rt/transports/http-undertow/src/main/java/org/apache/cxf/transport/http_undertow/spring/UndertowHTTPServerEngineFactoryBeanDefinitionParser.java b/rt/transports/http-undertow/src/main/java/org/apache/cxf/transport/http_undertow/spring/UndertowHTTPServerEngineFactoryBeanDefinitionParser.java
index 8d84863..bc6ffb9 100644
--- a/rt/transports/http-undertow/src/main/java/org/apache/cxf/transport/http_undertow/spring/UndertowHTTPServerEngineFactoryBeanDefinitionParser.java
+++ b/rt/transports/http-undertow/src/main/java/org/apache/cxf/transport/http_undertow/spring/UndertowHTTPServerEngineFactoryBeanDefinitionParser.java
@@ -96,7 +96,7 @@ public class UndertowHTTPServerEngineFactoryBeanDefinitionParser
// parser the engine list
List<Object> list =
getRequiredElementsList(element, ctx, new QName(HTTP_UNDERTOW_NS, "engine"), bean);
- if (list.size() > 0) {
+ if (!list.isEmpty()) {
bean.addPropertyValue("enginesList", list);
}
} catch (Exception e) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Cookies.java
----------------------------------------------------------------------
diff --git a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Cookies.java b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Cookies.java
index 35ab7cc..8c2708b 100644
--- a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Cookies.java
+++ b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Cookies.java
@@ -51,7 +51,7 @@ public class Cookies {
maintainSession = MessageUtils.getContextualBoolean(message, Message.MAINTAIN_SESSION, false);
//If we have any cookies and we are maintaining sessions, then use them
- if (maintainSession && sessionCookies.size() > 0) {
+ if (maintainSession && !sessionCookies.isEmpty()) {
new Headers(message).writeSessionCookies(sessionCookies);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
----------------------------------------------------------------------
diff --git a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
index a7c2e6f..10b53fc 100644
--- a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
+++ b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
@@ -570,7 +570,7 @@ public abstract class HTTPConduit
return true;
} else if (!HTTP_GET_METHOD.equals(httpMethod)) {
MessageContentsList objs = MessageContentsList.getContentsList(message);
- if (objs != null && objs.size() > 0) {
+ if (objs != null && !objs.isEmpty()) {
Object obj = objs.get(0);
if (obj.getClass() != String.class
|| (obj.getClass() == String.class && ((String)obj).length() > 0)) {
@@ -997,7 +997,7 @@ public abstract class HTTPConduit
for (Map.Entry<String, List<String>> head : headers.entrySet()) {
if ("Location".equalsIgnoreCase(head.getKey())) {
List<String> locs = head.getValue();
- if (locs != null && locs.size() > 0) {
+ if (locs != null && !locs.isEmpty()) {
String location = locs.get(0);
if (location != null) {
return location;
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSMessageUtils.java
----------------------------------------------------------------------
diff --git a/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSMessageUtils.java b/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSMessageUtils.java
index d8d227f..7f18f95 100644
--- a/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSMessageUtils.java
+++ b/rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/JMSMessageUtils.java
@@ -505,7 +505,7 @@ final class JMSMessageUtils {
if (headers != null) {
List<String> action = headers.remove("SOAPAction");
- if (action != null && action.size() > 0) {
+ if (action != null && !action.isEmpty()) {
soapAction = action.get(0);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
----------------------------------------------------------------------
diff --git a/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java b/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
index fb25582..4d7ad7e 100644
--- a/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
+++ b/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
@@ -525,7 +525,7 @@ public class MAPAggregatorImpl extends MAPAggregator {
if (aim == null || aim.size() == 0) {
missingWsaHeader = true;
}
- if (aim != null && aim.size() > 0) {
+ if (aim != null && !aim.isEmpty()) {
missingWsaHeader = true;
QName[] types = new QName[] {
MetadataConstants.ADDRESSING_ASSERTION_QNAME,
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/RMManager.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/RMManager.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/RMManager.java
index 3e77e7f..f84db06 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/RMManager.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/RMManager.java
@@ -504,7 +504,7 @@ public class RMManager {
@PreDestroy
public void shutdown() {
// shutdown remaining endpoints
- if (reliableEndpoints.size() > 0) {
+ if (!reliableEndpoints.isEmpty()) {
LOG.log(Level.FINE,
"Shutting down RMManager with {0} remaining endpoints.",
new Object[] {Integer.valueOf(reliableEndpoints.size())});
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/SequenceMonitor.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/SequenceMonitor.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/SequenceMonitor.java
index 6562ae4..e9b887b 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/SequenceMonitor.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/SequenceMonitor.java
@@ -60,7 +60,7 @@ public class SequenceMonitor {
}
public synchronized long getLastArrivalTime() {
- if (receiveTimes.size() > 0) {
+ if (!receiveTimes.isEmpty()) {
return receiveTimes.get(receiveTimes.size() - 1).longValue();
}
return 0;
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/persistence/jdbc/RMTxStore.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/persistence/jdbc/RMTxStore.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/persistence/jdbc/RMTxStore.java
index 1635e1c..f99809a 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/persistence/jdbc/RMTxStore.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/persistence/jdbc/RMTxStore.java
@@ -905,7 +905,7 @@ public class RMTxStore implements RMStore {
newCols.add(col);
}
}
- if (newCols.size() > 0) {
+ if (!newCols.isEmpty()) {
// need to add the new columns
if (LOG.isLoggable(Level.FINE)) {
LOG.log(Level.FINE, "Table " + tableName + " needs additional columns");
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RMSoapInInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RMSoapInInterceptor.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RMSoapInInterceptor.java
index bd401a2..8ced4c4 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RMSoapInInterceptor.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RMSoapInInterceptor.java
@@ -202,10 +202,10 @@ public class RMSoapInInterceptor extends AbstractSoapInterceptor {
}
}
}
- if (acks.size() > 0) {
+ if (!acks.isEmpty()) {
rmps.setAcks(acks);
}
- if (requested.size() > 0) {
+ if (!requested.isEmpty()) {
rmps.setAcksRequested(requested);
}
} catch (JAXBException ex) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
index d2518fb..d0e7018 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RedeliveryQueueImpl.java
@@ -136,7 +136,7 @@ public class RedeliveryQueueImpl implements RedeliveryQueue {
}
LOG.fine("Completed purging redeliver candidates.");
}
- if (purged.size() > 0) {
+ if (!purged.isEmpty()) {
RMStore store = manager.getStore();
if (null != store) {
store.removeMessages(seq.getIdentifier(), purged, false);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java
index 11d319a..cf6e3b8 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/soap/RetransmissionQueueImpl.java
@@ -199,7 +199,7 @@ public class RetransmissionQueueImpl implements RetransmissionQueue {
}
LOG.fine("Completed purging resend candidates.");
}
- if (purged.size() > 0) {
+ if (!purged.isEmpty()) {
RMStore store = manager.getStore();
if (null != store) {
store.removeMessages(sid, purged, true);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/IssuedTokenInterceptorProvider.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/IssuedTokenInterceptorProvider.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/IssuedTokenInterceptorProvider.java
index 33c2932..a1c794e 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/IssuedTokenInterceptorProvider.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/IssuedTokenInterceptorProvider.java
@@ -174,7 +174,7 @@ public class IssuedTokenInterceptorProvider extends AbstractPolicyInterceptorPro
message.getExchange().remove(SecurityConstants.TOKEN);
List<WSHandlerResult> results =
CastUtils.cast((List<?>)message.get(WSHandlerConstants.RECV_RESULTS));
- if (results != null && results.size() > 0) {
+ if (results != null && !results.isEmpty()) {
parseHandlerResults(results.get(0), message, ais);
}
} else {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/KerberosTokenInterceptorProvider.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/KerberosTokenInterceptorProvider.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/KerberosTokenInterceptorProvider.java
index b862b9c..9bd568e 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/KerberosTokenInterceptorProvider.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/interceptors/KerberosTokenInterceptorProvider.java
@@ -173,7 +173,7 @@ public class KerberosTokenInterceptorProvider extends AbstractPolicyInterceptorP
if (!isRequestor(message)) {
List<WSHandlerResult> results =
CastUtils.cast((List<?>)message.get(WSHandlerConstants.RECV_RESULTS));
- if (results != null && results.size() > 0) {
+ if (results != null && !results.isEmpty()) {
parseHandlerResults(results.get(0), message, aim, ais);
}
} else {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JOutInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JOutInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JOutInterceptor.java
index d6c6d71..ea11515 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JOutInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JOutInterceptor.java
@@ -205,7 +205,7 @@ public class PolicyBasedWSS4JOutInterceptor extends AbstractPhaseInterceptor<Soa
private boolean hasAttachments(SoapMessage mc) {
final Collection<org.apache.cxf.message.Attachment> attachments = mc.getAttachments();
- return attachments != null && attachments.size() > 0;
+ return attachments != null && !attachments.isEmpty();
}
public Set<String> getAfter() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JOutInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JOutInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JOutInterceptor.java
index 8a9d97d..43fd622 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JOutInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JOutInterceptor.java
@@ -293,7 +293,7 @@ public class WSS4JOutInterceptor extends AbstractWSS4JInterceptor {
private boolean hasAttachments(SoapMessage mc) {
final Collection<org.apache.cxf.message.Attachment> attachments = mc.getAttachments();
- return attachments != null && attachments.size() > 0;
+ return attachments != null && !attachments.isEmpty();
}
private void configureActions(SoapMessage mc, boolean doDebug,
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AbstractBindingBuilder.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AbstractBindingBuilder.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AbstractBindingBuilder.java
index 11f92f7..d515f8b 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AbstractBindingBuilder.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AbstractBindingBuilder.java
@@ -2182,7 +2182,7 @@ public abstract class AbstractBindingBuilder extends AbstractCommonBindingHandle
// prepare a SignatureConfirmation token
WSSecSignatureConfirmation wsc = new WSSecSignatureConfirmation(secHeader);
wsc.setIdAllocator(wssConfig.getIdAllocator());
- if (signatureActions.size() > 0) {
+ if (!signatureActions.isEmpty()) {
for (WSSecurityEngineResult wsr : signatureActions) {
byte[] sigVal = (byte[]) wsr.get(WSSecurityEngineResult.TAG_SIGNATURE_VALUE);
wsc.setSignatureValue(sigVal);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AsymmetricBindingHandler.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AsymmetricBindingHandler.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AsymmetricBindingHandler.java
index 7fdda59..3ae627a 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AsymmetricBindingHandler.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/AsymmetricBindingHandler.java
@@ -326,7 +326,7 @@ public class AsymmetricBindingHandler extends AbstractBindingBuilder {
}
WSSecBase encrBase = null;
- if (encryptionToken != null && encrParts.size() > 0) {
+ if (encryptionToken != null && !encrParts.isEmpty()) {
encrBase = doEncryption(wrapper, encrParts, true);
handleEncryptedSignedHeaders(encrParts, sigParts);
}
@@ -336,7 +336,7 @@ public class AsymmetricBindingHandler extends AbstractBindingBuilder {
}
try {
- if (sigParts.size() > 0) {
+ if (!sigParts.isEmpty()) {
if (initiatorWrapper != null && isRequestor()) {
doSignature(initiatorWrapper, sigParts, attached);
} else if (!isRequestor()) {
@@ -437,7 +437,7 @@ public class AsymmetricBindingHandler extends AbstractBindingBuilder {
List<WSEncryptionPart> encrParts,
boolean externalRef) {
//Do encryption
- if (recToken != null && recToken.getToken() != null && encrParts.size() > 0) {
+ if (recToken != null && recToken.getToken() != null && !encrParts.isEmpty()) {
AbstractToken encrToken = recToken.getToken();
assertPolicy(recToken);
assertPolicy(encrToken);
http://git-wip-us.apache.org/repos/asf/cxf/blob/003fb1e7/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/StaxAsymmetricBindingHandler.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/StaxAsymmetricBindingHandler.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/StaxAsymmetricBindingHandler.java
index efd7203..037a2cb 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/StaxAsymmetricBindingHandler.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/StaxAsymmetricBindingHandler.java
@@ -173,7 +173,7 @@ public class StaxAsymmetricBindingHandler extends AbstractStaxBindingHandler {
assertTokenWrapper(recipientSignatureToken);
assertToken(recipientSignatureToken.getToken());
}
- if (recipientSignatureToken != null && sigs.size() > 0) {
+ if (recipientSignatureToken != null && !sigs.isEmpty()) {
doSignature(recipientSignatureToken, sigs);
}
}
@@ -289,7 +289,7 @@ public class StaxAsymmetricBindingHandler extends AbstractStaxBindingHandler {
addSupportingTokens();
- if (encryptionToken != null && encrParts.size() > 0) {
+ if (encryptionToken != null && !encrParts.isEmpty()) {
if (isRequestor()) {
encrParts.addAll(encryptedTokensList);
} else {
@@ -319,7 +319,7 @@ public class StaxAsymmetricBindingHandler extends AbstractStaxBindingHandler {
sigParts.add(part);
}
- if (sigParts.size() > 0) {
+ if (!sigParts.isEmpty()) {
if (initiatorWrapper != null && isRequestor()) {
doSignature(initiatorWrapper, sigParts);
} else if (!isRequestor()) {
@@ -350,7 +350,7 @@ public class StaxAsymmetricBindingHandler extends AbstractStaxBindingHandler {
List<SecurePart> encrParts,
boolean externalRef) throws SOAPException {
//Do encryption
- if (recToken != null && recToken.getToken() != null && encrParts.size() > 0) {
+ if (recToken != null && recToken.getToken() != null && !encrParts.isEmpty()) {
AbstractToken encrToken = recToken.getToken();
AlgorithmSuite algorithmSuite = abinding.getAlgorithmSuite();