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


Reply via email to