You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by st...@apache.org on 2003/01/01 05:07:40 UTC

cvs commit: xml-axis/java/tools/org/apache/axis/tools/ant/wsdl Java2WsdlAntTask.java

stevel      2002/12/31 20:07:40

  Modified:    java/tools/org/apache/axis/tools/ant/wsdl
                        Java2WsdlAntTask.java
  Log:
  fix NPE reported on axis-user, where classpath defined and extraClasses undefined caused the NPE. Fix is just a check first
  
  Revision  Changes    Path
  1.15      +8 -4      xml-axis/java/tools/org/apache/axis/tools/ant/wsdl/Java2WsdlAntTask.java
  
  Index: Java2WsdlAntTask.java
  ===================================================================
  RCS file: /home/cvs/xml-axis/java/tools/org/apache/axis/tools/ant/wsdl/Java2WsdlAntTask.java,v
  retrieving revision 1.14
  retrieving revision 1.15
  diff -u -r1.14 -r1.15
  --- Java2WsdlAntTask.java	11 Dec 2002 22:42:53 -0000	1.14
  +++ Java2WsdlAntTask.java	1 Jan 2003 04:07:40 -0000	1.15
  @@ -173,10 +173,14 @@
                 Project.MSG_VERBOSE);
               ClassUtils.setClassLoader(className, cl);
               ClassUtils.setClassLoader(implClass, cl);
  -            StringTokenizer tokenizer = new StringTokenizer(extraClasses, " ,");
  -            while (tokenizer.hasMoreTokens()) {
  -                String clsName = tokenizer.nextToken();
  -                ClassUtils.setClassLoader(clsName, cl);
  +            //add extra classes to the classpath when the classpath attr is not null
  +            //??why do we do this? Why don't we do this when the classpath is unset?
  +            if(extraClasses!=null) {
  +                StringTokenizer tokenizer = new StringTokenizer(extraClasses, " ,");
  +                while (tokenizer.hasMoreTokens()) {
  +                    String clsName = tokenizer.nextToken();
  +                    ClassUtils.setClassLoader(clsName, cl);
  +                }
               }
           }
           try {