Just curious. What was the point of this? The imports used to be in alphabetical order. Now I have no clue what the order is. I don't mind seeing imports cleaned up, but this just rearranged things (and pulled out a few blank lines, which I personally like).

It might be helpful if we could decide on a convention for imports, so that when we're in the mood, we can fix our code to match the convention. But just rearranging things without explanation isn't very useful.

Russell Butek
[EMAIL PROTECTED]

Please respond to [EMAIL PROTECTED]

To: [EMAIL PROTECTED]
cc:
Subject: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/symbolTable Utils.java



tomj        2002/06/02 19:03:32

    Modified:    java/src/org/apache/axis/wsdl/symbolTable Utils.java
    Log:
    optimze imports


    Revision  Changes    Path
    1.7       +6 -10     xml-axis/java/src/org/apache/axis/wsdl/symbolTable/Utils.java


    Index: Utils.java
    ===================================================================
    RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/symbolTable/Utils.java,v
    retrieving revision 1.6
    retrieving revision 1.7
    diff -u -r1.6 -r1.7
    --- Utils.java 31 May 2002 19:08:10 -0000 1.6
    +++ Utils.java 3 Jun 2002 02:03:32 -0000 1.7
    @@ -54,20 +54,16 @@

    */
    package org.apache.axis.wsdl.symbolTable;


    -import java.util.HashSet;
    -import java.util.Iterator;
    -import java.util.Vector;
    +import org.apache.axis.Constants;
    +import org.w3c.dom.NamedNodeMap;
    +import org.w3c.dom.Node;


    import javax.wsdl.QName;
    -

    import javax.xml.rpc.holders.BooleanHolder;
    import javax.xml.rpc.holders.IntHolder;

    -
    -import org.apache.axis.Constants;
    -
    -import org.w3c.dom.NamedNodeMap;
    -
    -import org.w3c.dom.Node;
    +import java.util.HashSet;
    +import java.util.Iterator;
    +import java.util.Vector;


    /**
    * This class contains static utility methods for the emitter.



Reply via email to