You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/08/15 23:13:02 UTC

svn commit: r1373631 - /openejb/trunk/openejb/server/openejb-cxf/src/main/java/org/apache/openejb/server/cxf/CxfEndpoint.java

Author: rmannibucau
Date: Wed Aug 15 21:13:02 2012
New Revision: 1373631

URL: http://svn.apache.org/viewvc?rev=1373631&view=rev
Log:
no need to read twice the same value, in parcitular when it can be a bit long

Modified:
    openejb/trunk/openejb/server/openejb-cxf/src/main/java/org/apache/openejb/server/cxf/CxfEndpoint.java

Modified: openejb/trunk/openejb/server/openejb-cxf/src/main/java/org/apache/openejb/server/cxf/CxfEndpoint.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/server/openejb-cxf/src/main/java/org/apache/openejb/server/cxf/CxfEndpoint.java?rev=1373631&r1=1373630&r2=1373631&view=diff
==============================================================================
--- openejb/trunk/openejb/server/openejb-cxf/src/main/java/org/apache/openejb/server/cxf/CxfEndpoint.java (original)
+++ openejb/trunk/openejb/server/openejb-cxf/src/main/java/org/apache/openejb/server/cxf/CxfEndpoint.java Wed Aug 15 21:13:02 2012
@@ -149,10 +149,13 @@ public abstract class CxfEndpoint {
 		svrFactory.setStart(false);
 		svrFactory.setServiceBean(implementor);
         svrFactory.setDestinationFactory(httpTransportFactory);
-        if (getEndpointProperties() != null) {
-            svrFactory.setProperties(getEndpointProperties());
+
+        final Map<String, Object> properties = getEndpointProperties();
+        if (properties != null) {
+            svrFactory.setProperties(properties);
         }
 
+
 		if (HTTPBinding.HTTP_BINDING.equals(implInfo.getBindingType())) {
 			svrFactory.setTransportId("http://cxf.apache.org/bindings/xformat");
 		}