axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Davanum Srinivas <dava...@gmail.com>
Subject [Axis2] ChainedParamReader (Re: svn commit: r358887 - /webservices/axis2/trunk/java/modules/wsdl/src/org/apache/axis2/wsdl/java2wsdl/bytecode/ChainedParamReader.java)
Date Sat, 24 Dec 2005 02:33:50 GMT
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, dims@apache.org <dims@apache.org> 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/

Mime
View raw message