Mailing-List: contact cxf-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cxf-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of glen.mazza@verizon.net
 designates 206.46.252.42 as permitted sender)
Date: Wed, 01 Aug 2007 15:28:55 -0400
From: Glen Mazza <glen.mazza@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
In-reply-to: <200708011500.47247.dkulp@apache.org>
To: cxf-dev@incubator.apache.org
Message-id: <1185996535.785.0.camel@gmazza-desktop>
MIME-version: 1.0
Content-type: text/plain
Content-transfer-encoding: 7bit
References: <20070801170453.BE9DA1A981C@eris.apache.org>
	<1185990267.27438.6.camel@gmazza-desktop>
	<200708011500.47247.dkulp@apache.org>

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.
> 
>