Hi Dims;
Yes , I will do . What I did was I just copied the code from Axis1.x and did not go deep into the code. And I tried to add few tast case as well.

Thanks,
Deepal
................................................................
~Future is Open~

----- Original Message ----- From: "Davanum Srinivas" <[EMAIL PROTECTED]>
To: "Axis developer list" <[email protected]>
Sent: Saturday, December 24, 2005 8:33 AM
Subject: [Axis2] ChainedParamReader (Re: svn commit: r358887 - /webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java)


Deepal,

think we need to revisit this class. I just realized the recursion is
because the debug info is not present in the compiled classes and it
just went into a loop. My change will make sure it will go up the
chain to a java.lang.Object.class but am not sure if we should...could
u please double-check the correct bhavior with and without debug info?

thanks,
dims

On 12/23/05, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
Author: dims
Date: Fri Dec 23 16:07:48 2005
New Revision: 358887

URL: http://svn.apache.org/viewcvs?rev=358887&view=rev
Log:
fix recursion look up the class hierarchy

Modified:

webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java

Modified: webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java?rev=358887&r1=358886&r2=358887&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java (original) +++ webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java Fri Dec 23 16:07:48 2005
@@ -75,7 +75,7 @@
         }
         //if we here, it means we need to create new chain.
         Class cls = (Class) clsChain.get(chain.size() - 1);
-        while (cls.getSuperclass() != null) {
+ while (cls != null && cls != java.lang.Object.class && cls.getSuperclass() != null) {
             Class superClass = cls.getSuperclass();
             try {
                 ParamReader _reader = new ParamReader(superClass);
@@ -90,6 +90,7 @@
//can not find the super class in the class path, abort here
                 return null;
             }
+            cls = superClass;
         }
         methodToParamMap.put(method, ret);
         return null;





--
Davanum Srinivas : http://wso2.com/blogs/


Reply via email to