You are viewing a plain text version of this content. The canonical link for it is here.
Posted to rpc-dev@xml.apache.org by dl...@apache.org on 2002/08/07 19:35:39 UTC

cvs commit: xml-rpc/src/java/org/apache/xmlrpc/applet SimpleXmlRpcClient.java

dlr         2002/08/07 10:35:39

  Modified:    src/java/org/apache/xmlrpc/applet SimpleXmlRpcClient.java
  Log:
  Added some notes.
  
  Revision  Changes    Path
  1.5       +5 -4      xml-rpc/src/java/org/apache/xmlrpc/applet/SimpleXmlRpcClient.java
  
  Index: SimpleXmlRpcClient.java
  ===================================================================
  RCS file: /home/cvs/xml-rpc/src/java/org/apache/xmlrpc/applet/SimpleXmlRpcClient.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -u -r1.4 -r1.5
  --- SimpleXmlRpcClient.java	20 Mar 2002 15:11:03 -0000	1.4
  +++ SimpleXmlRpcClient.java	7 Aug 2002 17:35:39 -0000	1.5
  @@ -80,13 +80,14 @@
   import uk.co.wilson.xml.MinML;
   
   /**
  - * A simple XML-RPC client
  + * A simple XML-RPC client.
  + *
  + * FIXME: This code is VERY out of date with the rest of the package.
    *
    * @version $Id$
    */
   public class SimpleXmlRpcClient
   {
  -
       URL url;
   
       /**
  @@ -130,7 +131,7 @@
   }
   
   /**
  - *
  + * FIXME: Leverage the XmlRpc class.
    */
   class XmlRpcSupport extends HandlerBase
   {
  
  
  

Re: cvs commit: xml-rpc/src/java/org/apache/xmlrpc/applet SimpleXmlRpcClient.java

Posted by Daniel Rall <dl...@finemaltcoding.com>.
Someone needs to step up and gut this class.  80% of the code in it is
duplicated in the core package.

dlr@apache.org writes:

> dlr         2002/08/07 10:35:39
> 
>   Modified:    src/java/org/apache/xmlrpc/applet SimpleXmlRpcClient.java
>   Log:
>   Added some notes.
>   
>   Revision  Changes    Path
>   1.5       +5 -4      xml-rpc/src/java/org/apache/xmlrpc/applet/SimpleXmlRpcClient.java
>   
>   Index: SimpleXmlRpcClient.java
>   ===================================================================
>   RCS file: /home/cvs/xml-rpc/src/java/org/apache/xmlrpc/applet/SimpleXmlRpcClient.java,v
>   retrieving revision 1.4
>   retrieving revision 1.5
>   diff -u -u -r1.4 -r1.5
>   --- SimpleXmlRpcClient.java	20 Mar 2002 15:11:03 -0000	1.4
>   +++ SimpleXmlRpcClient.java	7 Aug 2002 17:35:39 -0000	1.5
>   @@ -80,13 +80,14 @@
>    import uk.co.wilson.xml.MinML;
>    
>    /**
>   - * A simple XML-RPC client
>   + * A simple XML-RPC client.
>   + *
>   + * FIXME: This code is VERY out of date with the rest of the package.
>     *
>     * @version $Id$
>     */
>    public class SimpleXmlRpcClient
>    {
>   -
>        URL url;
>    
>        /**
>   @@ -130,7 +131,7 @@
>    }
>    
>    /**
>   - *
>   + * FIXME: Leverage the XmlRpc class.
>     */
>    class XmlRpcSupport extends HandlerBase
>    {

Re: cvs commit: xml-rpc/src/java/org/apache/xmlrpc/applet SimpleXmlRpcClient.java

Posted by Daniel Rall <dl...@finemaltcoding.com>.
Someone needs to step up and gut this class.  80% of the code in it is
duplicated in the core package.

dlr@apache.org writes:

> dlr         2002/08/07 10:35:39
> 
>   Modified:    src/java/org/apache/xmlrpc/applet SimpleXmlRpcClient.java
>   Log:
>   Added some notes.
>   
>   Revision  Changes    Path
>   1.5       +5 -4      xml-rpc/src/java/org/apache/xmlrpc/applet/SimpleXmlRpcClient.java
>   
>   Index: SimpleXmlRpcClient.java
>   ===================================================================
>   RCS file: /home/cvs/xml-rpc/src/java/org/apache/xmlrpc/applet/SimpleXmlRpcClient.java,v
>   retrieving revision 1.4
>   retrieving revision 1.5
>   diff -u -u -r1.4 -r1.5
>   --- SimpleXmlRpcClient.java	20 Mar 2002 15:11:03 -0000	1.4
>   +++ SimpleXmlRpcClient.java	7 Aug 2002 17:35:39 -0000	1.5
>   @@ -80,13 +80,14 @@
>    import uk.co.wilson.xml.MinML;
>    
>    /**
>   - * A simple XML-RPC client
>   + * A simple XML-RPC client.
>   + *
>   + * FIXME: This code is VERY out of date with the rest of the package.
>     *
>     * @version $Id$
>     */
>    public class SimpleXmlRpcClient
>    {
>   -
>        URL url;
>    
>        /**
>   @@ -130,7 +131,7 @@
>    }
>    
>    /**
>   - *
>   + * FIXME: Leverage the XmlRpc class.
>     */
>    class XmlRpcSupport extends HandlerBase
>    {