Dims already pointed out the switch-sense problem, and I've just fixed
it - am rerunning all-tests and will check in in a minute. :)

--G 

> -----Original Message-----
> From: Tom Jordahl [mailto:[EMAIL PROTECTED] 
> Sent: Wednesday, April 13, 2005 11:44 AM
> To: [EMAIL PROTECTED]; [email protected]
> Subject: RE: cvs commit: 
> ws-axis/java/tools/org/apache/axis/tools/ant/wsdl 
> Wsdl2javaAntTask.java
> 
> Sweet!
> 
> One thing, the WSDL2Java switch seems to turn OFF the unwrapping of
> arrays when it is present.  That doesn't seem right.
> 
>   +            case UNWRAP_ARRAYS_OPT:
>   +                emitter.setUnwrapArrays(false);
>   +                break;
>   +
> 
> Did you want the switch to be an on/off switch?  Then you 
> should have a
> switch:
>   -noUnwrapArrays
> 
> Or it should take a mandatory argument:
>   -unwrapArrays on/off
> 
> 
> As it is, the presence of the positive switch means a negative thing.
> 
> Oh and also you should update the docs for WSDL2Java to explain what
> this does to people. :-)
> 
> --
> Tom Jordahl
> Macromedia Server Development
> 
> > -----Original Message-----
> > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> > Sent: Tuesday, April 12, 2005 6:18 PM
> > To: [EMAIL PROTECTED]
> > Subject: cvs commit: 
> ws-axis/java/tools/org/apache/axis/tools/ant/wsdl
> > Wsdl2javaAntTask.java
> > 
> > gdaniels    2005/04/12 15:18:17
> > 
> >   Modified:    java/src/org/apache/axis/client Call.java
> >                java/src/org/apache/axis/description ElementDesc.java
> >                         ParameterDesc.java
> >                java/src/org/apache/axis/encoding
> SerializationContext.java
> >                java/src/org/apache/axis/encoding/ser
> > ArrayDeserializer.java
> >                         ArraySerializer.java BeanDeserializer.java
> >                         BeanSerializer.java
> >                java/src/org/apache/axis/i18n resource.properties
> >                java/src/org/apache/axis/message RPCElement.java
> >                         RPCHandler.java RPCParam.java
> >                java/src/org/apache/axis/utils
> BeanPropertyDescriptor.java
> >                         FieldPropertyDescriptor.java
> >                java/src/org/apache/axis/wsdl WSDL2Java.java
> >                java/src/org/apache/axis/wsdl/fromJava Emitter.java
> >                         Types.java
> >                java/src/org/apache/axis/wsdl/gen Parser.java
> >                java/src/org/apache/axis/wsdl/symbolTable
> ElementDecl.java
> >                         SchemaUtils.java SymbolTable.java
> TypeEntry.java
> >                java/src/org/apache/axis/wsdl/toJava Emitter.java
> >                         JavaBeanHelperWriter.java 
> JavaStubWriter.java
> >                java/test/wsdl/arrays3
> >                         AddrBookServiceImplServiceTestCase.java
> >                java/test/wsdl/clash build.xml
> >                java/test/wsdl/doclit_arrays deploy.wsdd
> >                java/test/wsdl/interop4/groupH/simpleDocLit build.xml
> >                java/test/wsdl/marshall2 build.xml
> >                java/test/wsdl/qnameser build.xml
> >                java/test/wsdl/roundtrip build.xml
> >                java/tools/org/apache/axis/tools/ant/wsdl
> >                         Wsdl2javaAntTask.java
> >   Log:
> >   Support "wrapped" literal arrays, of the following form:
>  
> 
> 

Reply via email to