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