cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Glen Mazza <glen.ma...@verizon.net>
Subject Re: svn commit: r561871 - in /incubator/cxf/trunk/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators: AntGenerator.java ClientGenerator.java ServerGenerator.java
Date Wed, 01 Aug 2007 19:28:55 GMT
Oops!  How could I have missed that?

Glen

Am Mittwoch, den 01.08.2007, 15:00 -0400 schrieb Daniel Kulp:
> Glen,
> 
> On Wednesday 01 August 2007 13:44, Glen Mazza wrote:
> > > +                String interfaceName = jp.getInterfaceClass();
> > > (A)                 JavaInterface intf =
> > > interfaces.get(interfaceName); (B)+                if (intf == null)
> > > {
> > > (C)+                    interfaceName = jp.getPortType();
> > > (D)+                    intf = interfaces.get(interfaceName);
> > > (E)+                }
> >
> > There may be a bug here (and in the two cases below).  Namely, I can't
> > see the reason for statement (D) above.  If (A) results in intf being
> > set to NULL, lines (B) - (E) will be activated.  But how can statement
> > (D) set intf to anything but null again--statement (C) has no side
> > effect in adding interface names to the interfaces object.
> 
> Line (C) results in a new value for interfaceName which results in a new 
> value being used to lookup stuff from the interfaces Map.   The map 
> doesn't change, the lookup key does.
> 
> 


Mime
View raw message