You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ni...@apache.org on 2013/05/21 06:13:29 UTC
svn commit: r1484660 - in /cxf/trunk/rt/transports/http-jetty/src:
main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java
Author: ningjiang
Date: Tue May 21 04:13:29 2013
New Revision: 1484660
URL: http://svn.apache.org/r1484660
Log:
polish the code of JettyHTTPDestination
Modified:
cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
cxf/trunk/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java
Modified: cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java?rev=1484660&r1=1484659&r2=1484660&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java (original)
+++ cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java Tue May 21 04:13:29 2013
@@ -85,13 +85,13 @@ public class JettyHTTPDestination extend
private boolean configFinalized;
/**
- * Constructor, using Jetty server engine.
- *
- * @param b the associated Bus
- * @param ci the associated conduit initiator
- * @param endpointInfo the endpoint info of the destination
- * @param serverEngineFactory
- * @throws IOException
+ * Constructor
+ *
+ * @param b the associated Bus
+ * @param registry the associated destinationRegistry
+ * @param ei the endpoint info of the destination
+ * @param serverEngineFactory the serverEngineFactory which could be used to create ServerEngine
+ * @throws java.io.IOException
*/
public JettyHTTPDestination(
Bus bus,
@@ -99,7 +99,7 @@ public class JettyHTTPDestination extend
EndpointInfo ei,
JettyHTTPServerEngineFactory serverEngineFactory
) throws IOException {
- //Add the defualt port if the address is missing it
+ //Add the default port if the address is missing it
super(bus, registry, ei, getAddressValue(ei, true).getAddress(), true);
this.serverEngineFactory = serverEngineFactory;
nurl = new URL(endpointInfo.getAddress());
Modified: cxf/trunk/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java?rev=1484660&r1=1484659&r2=1484660&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java (original)
+++ cxf/trunk/rt/transports/http-jetty/src/test/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngineTest.java Tue May 21 04:13:29 2013
@@ -99,7 +99,7 @@ public class JettyHTTPServerEngineTest e
"Engine references for the same port should point to the same instance",
engine == factory.retrieveJettyHTTPServerEngine(PORT1));
- JettyHTTPServerEngineFactory.destroyForPort(1234);
+ JettyHTTPServerEngineFactory.destroyForPort(PORT1);
}
@Test