ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jgl...@apache.org
Subject svn commit: r1504315 - in /ant/core/trunk: ./ manual/ src/main/org/apache/tools/ant/ src/main/org/apache/tools/ant/helper/ src/main/org/apache/tools/ant/launch/ src/main/org/apache/tools/ant/taskdefs/ src/main/org/apache/tools/ant/taskdefs/compilers/ s...
Date Wed, 17 Jul 2013 22:46:11 GMT
Author: jglick
Date: Wed Jul 17 22:46:10 2013
New Revision: 1504315

URL: http://svn.apache.org/r1504315
Log:
sconnolly advises using TODO rather than XXX in comments.

Modified:
    ant/core/trunk/build.xml
    ant/core/trunk/manual/properties.html
    ant/core/trunk/manual/tutorial-tasks-filesets-properties.html
    ant/core/trunk/src/main/org/apache/tools/ant/AntClassLoader.java
    ant/core/trunk/src/main/org/apache/tools/ant/Main.java
    ant/core/trunk/src/main/org/apache/tools/ant/Task.java
    ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelper2.java
    ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelperImpl.java
    ant/core/trunk/src/main/org/apache/tools/ant/launch/Locator.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractCvsTask.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractJarSignerTask.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AntStructure.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Classloader.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ExecuteJava.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Jar.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/JikesOutputParser.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/compilers/Jikes.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jsp/WLJspc.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java
    ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTaskMirrorImpl.java
    ant/core/trunk/src/main/org/apache/tools/ant/types/Path.java
    ant/core/trunk/src/main/org/apache/tools/ant/types/resources/Union.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroRegexp.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaRegexpRegexp.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/Jdk14RegexpRegexp.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java
    ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java
    ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/ExecTaskTest.java
    ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/InputTest.java
    ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/StyleTest.java
    ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/optional/XsltTest.java
    ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/DeweyDecimalTest.java

Modified: ant/core/trunk/build.xml
URL: http://svn.apache.org/viewvc/ant/core/trunk/build.xml?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/build.xml (original)
+++ ant/core/trunk/build.xml Wed Jul 17 22:46:10 2013
@@ -567,7 +567,7 @@
             <selector refid="not.in.kaffe" if="kaffe"/>
 
             <selector refid="needs.apache-resolver" unless="apache.resolver.present"/>
-            <selector refid="needs.junit" unless="junit.present"/> <!-- XXX should
perhaps use -source 1.4? -->
+            <selector refid="needs.junit" unless="junit.present"/> <!-- TODO should
perhaps use -source 1.4? -->
             <selector refid="needs.junit4" unless="junit4.present"/>
             <selector refid="needs.apache-regexp"
               unless="apache.regexp.present"/>
@@ -1631,7 +1631,7 @@ ${antunit.reports}
         <property name="test.junit.vmargs" value=""/>
         <property name="ant.junit.failureCollector"
                   value="${junit.collector.dir}/${junit.collector.class}"/>
-        <!-- XXX includeantruntime="false" does not solve "multiple versions of ant detected
in path for junit" warning -->
+        <!-- TODO includeantruntime="false" does not solve "multiple versions of ant detected
in path for junit" warning -->
         <junit printsummary="${junit.summary}"
                haltonfailure="${test.haltonfailure}"
                fork="${junit.fork}"

Modified: ant/core/trunk/manual/properties.html
URL: http://svn.apache.org/viewvc/ant/core/trunk/manual/properties.html?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/manual/properties.html (original)
+++ ant/core/trunk/manual/properties.html Wed Jul 17 22:46:10 2013
@@ -61,7 +61,7 @@
   </p>
 
   <p>In addition, Ant has some built-in properties:</p>
-<pre><!-- XXX use <dl><dt><code>...</code></dt><dd>...</dd></dl>
instead -->
+<pre><!-- TODO use <dl><dt><code>...</code></dt><dd>...</dd></dl>
instead -->
 basedir             the absolute path of the project's basedir (as set
                     with the basedir attribute of <a href="using.html#projects">&lt;project&gt;</a>).
 ant.file            the absolute path of the buildfile.

Modified: ant/core/trunk/manual/tutorial-tasks-filesets-properties.html
URL: http://svn.apache.org/viewvc/ant/core/trunk/manual/tutorial-tasks-filesets-properties.html?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/manual/tutorial-tasks-filesets-properties.html (original)
+++ ant/core/trunk/manual/tutorial-tasks-filesets-properties.html Wed Jul 17 22:46:10 2013
@@ -883,7 +883,7 @@ do a second checkstyle run. Now our task
 <h3>Creating the diff</h3>
 <p>Creating a diff for Ant is very easy: just start <tt>ant -f patch.xml</tt>
and all is done
 automatically. This step requires a cvs executable in your path and internet access (more
precise:
-cvs access). As a result we get a file <i> XXX </i>.</p>
+cvs access). As a result we get a file <i> TODO </i>.</p>
 -->
 
 

Modified: ant/core/trunk/src/main/org/apache/tools/ant/AntClassLoader.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/AntClassLoader.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/AntClassLoader.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/AntClassLoader.java Wed Jul 17 22:46:10 2013
@@ -824,7 +824,7 @@ public class AntClassLoader extends Clas
         // designated to use a specific loader first
         // (this one or the parent one)
 
-        // XXX - shouldn't this always return false in isolated mode?
+        // TODO - shouldn't this always return false in isolated mode?
 
         boolean useParentFirst = parentFirst;
 
@@ -1561,4 +1561,4 @@ public class AntClassLoader extends Clas
         return new AntClassLoader(parent, project, path, parentFirst);
     }
 
-}
\ No newline at end of file
+}

Modified: ant/core/trunk/src/main/org/apache/tools/ant/Main.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/Main.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/Main.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/Main.java Wed Jul 17 22:46:10 2013
@@ -962,7 +962,7 @@ public class Main implements AntMain {
         project.setInputHandler(handler);
     }
 
-    // XXX: (Jon Skeet) Any reason for writing a message and then using a bare
+    // TODO: (Jon Skeet) Any reason for writing a message and then using a bare
     // RuntimeException rather than just using a BuildException here? Is it
     // in case the message could end up being written to no loggers (as the
     // loggers could have failed to be created due to this failure)?

Modified: ant/core/trunk/src/main/org/apache/tools/ant/Task.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/Task.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/Task.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/Task.java Wed Jul 17 22:46:10 2013
@@ -185,7 +185,7 @@ public abstract class Task extends Proje
         this.wrapper = wrapper;
     }
 
-    // XXX: (Jon Skeet) The comment "if it hasn't been done already" may
+    // TODO: (Jon Skeet) The comment "if it hasn't been done already" may
     // not be strictly true. wrapper.maybeConfigure() won't configure the same
     // attributes/text more than once, but it may well add the children again,
     // unless I've missed something.

Modified: ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelper2.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelper2.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelper2.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelper2.java Wed Jul 17 22:46:10
2013
@@ -699,7 +699,7 @@ public class ProjectHelper2 extends Proj
             // Set the location of the implicit target associated with the project tag
             context.getImplicitTarget().setLocation(new Location(context.getLocator()));
 
-            /** XXX I really don't like this - the XML processor is still
+            /** TODO I really don't like this - the XML processor is still
              * too 'involved' in the processing. A better solution (IMO)
              * would be to create UE for Project and Target too, and
              * then process the tree and have Project/Target deal with
@@ -751,13 +751,13 @@ public class ProjectHelper2 extends Proj
                         baseDir = value;
                     }
                 } else {
-                    // XXX ignore attributes in a different NS ( maybe store them ? )
+                    // TODO ignore attributes in a different NS ( maybe store them ? )
                     throw new SAXParseException("Unexpected attribute \"" + attrs.getQName(i)
                                                 + "\"", context.getLocator());
                 }
             }
 
-            // XXX Move to Project ( so it is shared by all helpers )
+            // TODO Move to Project ( so it is shared by all helpers )
             String antFileProp =
                 MagicNames.ANT_FILE + "." + context.getCurrentProjectName();
             String dup = project.getProperty(antFileProp);

Modified: ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelperImpl.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelperImpl.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelperImpl.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/helper/ProjectHelperImpl.java Wed Jul 17
22:46:10 2013
@@ -727,7 +727,7 @@ public class ProjectHelperImpl extends P
             if (task == null) {
                 task = new UnknownElement(tag);
                 task.setProject(helperImpl.project);
-                //XXX task.setTaskType(tag);
+                //TODO task.setTaskType(tag);
                 task.setTaskName(tag);
             }
             task.setLocation(new Location(helperImpl.locator));

Modified: ant/core/trunk/src/main/org/apache/tools/ant/launch/Locator.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/launch/Locator.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/launch/Locator.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/launch/Locator.java Wed Jul 17 22:46:10 2013
@@ -166,7 +166,7 @@ public final class Locator {
      *
      * <p>Will be an absolute path if the given URI is absolute.</p>
      *
-     * <p>Prior to Java 1.4,<!-- XXX is JDK version actually relevant? -->
+     * <p>Prior to Java 1.4,<!-- TODO is JDK version actually relevant? -->
      * swallows '%' that are not followed by two characters.</p>
      *
      * See <a href="http://www.w3.org/TR/xml11/#dt-sysid">dt-sysid</a>
@@ -181,7 +181,7 @@ public final class Locator {
     public static String fromURI(String uri) {
         return fromURIJava13(uri);
         // #buzilla8031: first try Java 1.4.
-        // XXX should use java.net.URI now that we can rely on 1.4...
+        // TODO should use java.net.URI now that we can rely on 1.4...
         // but check for UNC-related regressions, e.g. #42275
         // (and remember that \\server\share\file -> file:////server/share/file
         // rather than -> file://server/share/file as it should;
@@ -392,7 +392,7 @@ public final class Locator {
      * Convert a File to a URL.
      * File.toURL() does not encode characters like #.
      * File.toURI() has been introduced in java 1.4, so
-     * Ant cannot use it (except by reflection) <!-- XXX no longer true -->
+     * Ant cannot use it (except by reflection) <!-- TODO no longer true -->
      * FileUtils.toURI() cannot be used by Locator.java
      * Implemented this way.
      * File.toURL() adds file: and changes '\' to '/' for dos OSes

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractCvsTask.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractCvsTask.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractCvsTask.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractCvsTask.java Wed Jul 17
22:46:10 2013
@@ -259,7 +259,7 @@ public abstract class AbstractCvsTask ex
      * @throws BuildException if failonError is set to true and the cvs command fails
      */
     protected void runCommand(Commandline toExecute) throws BuildException {
-        // XXX: we should use JCVS (www.ice.com/JCVS) instead of
+        // TODO: we should use JCVS (www.ice.com/JCVS) instead of
         // command line execution so that we don't rely on having
         // native CVS stuff around (SM)
 

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractJarSignerTask.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractJarSignerTask.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractJarSignerTask.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AbstractJarSignerTask.java Wed Jul
17 22:46:10 2013
@@ -262,7 +262,7 @@ public abstract class AbstractJarSignerT
             // Try to avoid showing password prompts on log output, as they would be confusing.
             LineContainsRegExp filter = new LineContainsRegExp();
             RegularExpression rx = new RegularExpression();
-            // XXX only handles English locale, not ja or zh_CN
+            // TODO only handles English locale, not ja or zh_CN
             rx.setPattern("^(Enter Passphrase for keystore: |Enter key password for .+: )$");
             filter.addConfiguredRegexp(rx);
             filter.setNegate(true);

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AntStructure.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AntStructure.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AntStructure.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/AntStructure.java Wed Jul 17 22:46:10
2013
@@ -288,7 +288,7 @@ public class AntStructure extends Task {
                 ih = IntrospectionHelper.getHelper(p, element);
             } catch (Throwable t) {
                 /*
-                 * XXX - failed to load the class properly.
+                 * TODO - failed to load the class properly.
                  *
                  * should we print a warning here?
                  */
@@ -433,7 +433,7 @@ public class AntStructure extends Task {
             final int length = s.length();
             for (int i = 0; i < length; i++) {
                 char c = s.charAt(i);
-                // XXX - we are committing CombiningChar and Extender here
+                // TODO - we are committing CombiningChar and Extender here
                 if (!Character.isLetterOrDigit(c)
                     && c != '.' && c != '-' && c != '_' &&
c != ':') {
                     return false;

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Classloader.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Classloader.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Classloader.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Classloader.java Wed Jul 17 22:46:10
2013
@@ -178,7 +178,7 @@ public class Classloader extends Task {
                 obj = null; // a new one will be created.
             }
 
-            // XXX maybe use reflection to addPathElement (other patterns ?)
+            // TODO maybe use reflection to addPathElement (other patterns ?)
             if (obj != null && !(obj instanceof AntClassLoader)) {
                 log("Referenced object is not an AntClassLoader",
                         Project.MSG_ERR);
@@ -235,7 +235,7 @@ public class Classloader extends Task {
                 }
             }
 
-            // XXX add exceptions
+            // TODO add exceptions
 
         } catch (Exception ex) {
             ex.printStackTrace();

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ExecuteJava.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ExecuteJava.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ExecuteJava.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/ExecuteJava.java Wed Jul 17 22:46:10
2013
@@ -154,7 +154,7 @@ public class ExecuteJava implements Runn
                 thread = new Thread(this, "ExecuteJava");
                 Task currentThreadTask
                     = project.getThreadTask(Thread.currentThread());
-                // XXX is the following really necessary? it is in the same thread group...
+                // TODO is the following really necessary? it is in the same thread group...
                 project.registerThreadTask(thread, currentThreadTask);
                 // if we run into a timeout, the run-away thread shall not
                 // make the VM run forever - if no timeout occurs, Ant's
@@ -188,7 +188,7 @@ public class ExecuteJava implements Runn
         } catch (SecurityException e) {
             throw e;
         } catch (ThreadDeath e) {
-            // XXX could perhaps also call thread.stop(); not sure if anyone cares
+            // TODO could perhaps also call thread.stop(); not sure if anyone cares
             throw e;
         } catch (Throwable e) {
             throw new BuildException(e);

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Jar.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Jar.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Jar.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Jar.java Wed Jul 17 22:46:10 2013
@@ -353,7 +353,7 @@ public class Jar extends Zip {
                 try {
                     zf.close();
                 } catch (IOException e) {
-                    // XXX - log an error?  throw an exception?
+                    // TODO - log an error?  throw an exception?
                 }
             }
         }
@@ -392,7 +392,7 @@ public class Jar extends Zip {
                 try {
                     zf.close();
                 } catch (IOException e) {
-                    // XXX - log an error?  throw an exception?
+                    // TODO - log an error?  throw an exception?
                 }
             }
         }

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java Wed Jul 17 22:46:10
2013
@@ -2212,7 +2212,7 @@ public class Javadoc extends Task {
         for (SourceFile sf : sourceFilesToDoc) {
             String sourceFileName = sf.getFile().getAbsolutePath();
             if (useExternalFile) {
-                // XXX what is the following doing?
+                // TODO what is the following doing?
                 //     should it run if !javadoc4 && executable != null?
                 if (sourceFileName.indexOf(" ") > -1) {
                     String name = sourceFileName;

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/JikesOutputParser.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/JikesOutputParser.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/JikesOutputParser.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/JikesOutputParser.java Wed Jul 17
22:46:10 2013
@@ -121,7 +121,7 @@ public class JikesOutputParser implement
         String line;
         String lower;
         // We assume, that every output, jikes does, stands for an error/warning
-        // XXX
+        // TODO
         // Is this correct?
 
         // TODO:

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/compilers/Jikes.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/compilers/Jikes.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/compilers/Jikes.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/compilers/Jikes.java Wed Jul 17
22:46:10 2013
@@ -162,7 +162,7 @@ public class Jikes extends DefaultCompil
 
     private void addPropertyParams(Commandline cmd) {
         /**
-         * XXX
+         * TODO
          * Perhaps we shouldn't use properties for these
          * three options (emacs mode, warnings and pedantic),
          * but include it in the javac directive?

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/Native2Ascii.java Wed Jul
17 22:46:10 2013
@@ -274,7 +274,7 @@ public class Native2Ascii extends Matchi
         }
 
         // Make intermediate directories if needed
-        // XXX JDK 1.1 doesn't have File.getParentFile,
+        // TODO JDK 1.1 doesn't have File.getParentFile,
         String parentName = destFile.getParent();
         if (parentName != null) {
             File parentFile = new File(parentName);

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jsp/WLJspc.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jsp/WLJspc.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jsp/WLJspc.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jsp/WLJspc.java Wed Jul
17 22:46:10 2013
@@ -148,7 +148,7 @@ public class WLJspc extends MatchingTask
         File jspFile = null;
         String parents = "";
         int j = 0;
-        //XXX  this array stuff is a remnant of prev trials.. gotta remove.
+        //TODO  this array stuff is a remnant of prev trials.. gotta remove.
         args[j++] = "-d";
         args[j++] = destinationDirectory.getAbsolutePath().trim();
         args[j++] = "-docroot";
@@ -169,7 +169,7 @@ public class WLJspc extends MatchingTask
 
         final int size = filesToDo.size();
         for (int i = 0; i < size; i++) {
-            //XXX
+            //TODO
             // All this to get package according to weblogic standards
             // Can be written better... this is too hacky!
             // Careful.. similar code in scanDir , but slightly different!!
@@ -271,7 +271,7 @@ public class WLJspc extends MatchingTask
         String pack = "";
         for (int i = 0; i < files.length; i++) {
             File srcFile = new File(this.sourceDirectory, files[i]);
-            //XXX
+            //TODO
             // All this to convert source to destination directory according
             // to weblogic standards Can be written better... this is too hacky!
             jspFile = new File(files[i]);

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java
(original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java
Wed Jul 17 22:46:10 2013
@@ -290,7 +290,7 @@ public class JUnitTestRunner implements 
                            boolean showOutput, boolean logTestListenerEvents,
                            ClassLoader loader) {
         super();
-        JUnitTestRunner.filtertrace = filtertrace; // XXX clumsy, should use instance field
somehow
+        JUnitTestRunner.filtertrace = filtertrace; // TODO clumsy, should use instance field
somehow
         this.junitTest = test;
         this.haltOnError = haltOnError;
         this.haltOnFailure = haltOnFailure;

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java Wed Jul 17
22:46:10 2013
@@ -2136,7 +2136,7 @@ public class FTP extends Task implements
         InputStream instream = null;
 
         try {
-            // XXX - why not simply new File(dir, filename)?
+            // TODO - why not simply new File(dir, filename)?
             File file = getProject().resolveFile(new File(dir, filename).getPath());
 
             if (newerOnly && isUpToDate(ftp, file, resolveFile(filename))) {

Modified: ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTaskMirrorImpl.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTaskMirrorImpl.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTaskMirrorImpl.java
(original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTaskMirrorImpl.java
Wed Jul 17 22:46:10 2013
@@ -1523,7 +1523,7 @@ public class FTPTaskMirrorImpl implement
         InputStream instream = null;
 
         try {
-            // XXX - why not simply new File(dir, filename)?
+            // TODO - why not simply new File(dir, filename)?
             File file = task.getProject().resolveFile(new File(dir, filename).getPath());
 
             if (task.isNewer() && isUpToDate(ftp, file, resolveFile(filename))) {

Modified: ant/core/trunk/src/main/org/apache/tools/ant/types/Path.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/Path.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/types/Path.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/types/Path.java Wed Jul 17 22:46:10 2013
@@ -601,7 +601,7 @@ public class Path extends DataType imple
         }
 
         if (System.getProperty("java.vendor").toLowerCase(Locale.ENGLISH).indexOf("microsoft")
>= 0) {
-            // XXX is this code still necessary? is there any 1.2+ port?
+            // TODO is this code still necessary? is there any 1.2+ port?
             // Pull in *.zip from packages directory
             FileSet msZipFiles = new FileSet();
             msZipFiles.setDir(new File(System.getProperty("java.home")

Modified: ant/core/trunk/src/main/org/apache/tools/ant/types/resources/Union.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/resources/Union.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/types/resources/Union.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/types/resources/Union.java Wed Jul 17 22:46:10
2013
@@ -116,7 +116,7 @@ public class Union extends BaseResourceC
      */
     @Deprecated
     @SuppressWarnings("unchecked")
-    protected <T> Collection<T> getCollection(boolean asString) { // XXX untypable
+    protected <T> Collection<T> getCollection(boolean asString) { // TODO untypable
         return asString ? (Collection<T>) getAllToStrings() : (Collection<T>)
getAllResources();
     }
 

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/LazyHashtable.java Wed Jul 17 22:46:10
2013
@@ -115,6 +115,6 @@ public class LazyHashtable extends Hasht
         return super.keys();
     }
 
-    // XXX Unfortunately JDK1.2 adds entrySet(), keySet(), values() -
+    // TODO Unfortunately JDK1.2 adds entrySet(), keySet(), values() -
     // implementing this requires a small hack, we can add it later.
 }

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/RegexpPatternMapper.java Wed Jul 17
22:46:10 2013
@@ -145,7 +145,7 @@ public class RegexpPatternMapper impleme
                         result.append(to[i]);
                     }
                 } else {
-                    // XXX - should throw an exception instead?
+                    // TODO - should throw an exception instead?
                     result.append('\\');
                 }
             } else {

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroRegexp.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroRegexp.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroRegexp.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaOroRegexp.java Wed Jul
17 22:46:10 2013
@@ -61,7 +61,7 @@ public class JakartaOroRegexp extends Ja
                         subst.append(c);
                     }
                 } else {
-                    // XXX - should throw an exception instead?
+                    // TODO - should throw an exception instead?
                     subst.append('\\');
                 }
             } else {

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaRegexpRegexp.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaRegexpRegexp.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaRegexpRegexp.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/JakartaRegexpRegexp.java Wed
Jul 17 22:46:10 2013
@@ -74,7 +74,7 @@ public class JakartaRegexpRegexp extends
                         result.append(c);
                     }
                 } else {
-                    // XXX - should throw an exception instead?
+                    // TODO - should throw an exception instead?
                     result.append('\\');
                 }
             } else {

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/Jdk14RegexpRegexp.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/Jdk14RegexpRegexp.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/Jdk14RegexpRegexp.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/Jdk14RegexpRegexp.java Wed Jul
17 22:46:10 2013
@@ -74,7 +74,7 @@ public class Jdk14RegexpRegexp extends J
                         subst.append(c);
                     }
                 } else {
-                    // XXX - should throw an exception instead?
+                    // TODO - should throw an exception instead?
                     subst.append('\\');
                 }
             } else {

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpFactory.java Wed Jul 17
22:46:10 2013
@@ -60,7 +60,7 @@ public class RegexpFactory extends Regex
 
         if (systemDefault != null) {
             return createRegexpInstance(systemDefault);
-            // XXX     should we silently catch possible exceptions and try to
+            // TODO     should we silently catch possible exceptions and try to
             //         load a different implementation?
         }
 

Modified: ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java (original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/util/regexp/RegexpMatcherFactory.java Wed
Jul 17 22:46:10 2013
@@ -63,7 +63,7 @@ public class RegexpMatcherFactory {
 
         if (systemDefault != null) {
             return createInstance(systemDefault);
-            // XXX     should we silently catch possible exceptions and try to
+            // TODO     should we silently catch possible exceptions and try to
             //         load a different implementation?
         }
 

Modified: ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/ExecTaskTest.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/ExecTaskTest.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/ExecTaskTest.java (original)
+++ ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/ExecTaskTest.java Wed Jul
17 22:46:10 2013
@@ -104,7 +104,7 @@ public class ExecTaskTest extends BuildF
         assertTrue("log file found after spawn", logFile.exists());
     }
 
-    /* XXX #50507 - fails at least on Linux
+    /* TODO #50507 - fails at least on Linux
     public void testOutAndErr() {
         project.executeTarget("test-out-and-err");
     }

Modified: ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/InputTest.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/InputTest.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/InputTest.java (original)
+++ ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/InputTest.java Wed Jul 17
22:46:10 2013
@@ -100,7 +100,7 @@ public class InputTest extends BuildFile
     }
 
     private String getKey(String key) {
-        return key; // XXX what is this for?
+        return key; // TODO what is this for?
     }
 
 }

Modified: ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/StyleTest.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/StyleTest.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/StyleTest.java (original)
+++ ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/StyleTest.java Wed Jul 17
22:46:10 2013
@@ -28,7 +28,7 @@ import org.apache.tools.ant.util.FileUti
 
 /**
  * TestCases for {@link XSLTProcess} task.
- * XXX merge with {@link XsltTest}?
+ * TODO merge with {@link XsltTest}?
  * @version 2003-08-05
  */
 public class StyleTest extends BuildFileTest {

Modified: ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/optional/XsltTest.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/optional/XsltTest.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/optional/XsltTest.java (original)
+++ ant/core/trunk/src/tests/junit/org/apache/tools/ant/taskdefs/optional/XsltTest.java Wed
Jul 17 22:46:10 2013
@@ -21,7 +21,7 @@ import org.apache.tools.ant.BuildFileTes
 
 /**
  * Tests the {@link XSLTProcess} task.
- * XXX merge with {@link StyleTest}?
+ * TODO merge with {@link StyleTest}?
  * @since Ant 1.5
  */
 public class XsltTest extends BuildFileTest {

Modified: ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/DeweyDecimalTest.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/DeweyDecimalTest.java?rev=1504315&r1=1504314&r2=1504315&view=diff
==============================================================================
--- ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/DeweyDecimalTest.java (original)
+++ ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/DeweyDecimalTest.java Wed Jul
17 22:46:10 2013
@@ -40,7 +40,7 @@ public class DeweyDecimalTest {
         new DeweyDecimal("1.2.");
     }
 
-    // XXX initial dots, empty string, null, negative numbers, ...
+    // TODO initial dots, empty string, null, negative numbers, ...
 
     @Test public void testHashCode() {
         assertEquals(new DeweyDecimal("1.2.3").hashCode(), new DeweyDecimal("1.2.3").hashCode());
@@ -63,6 +63,6 @@ public class DeweyDecimalTest {
         assertTrue(new DeweyDecimal("1.2").compareTo(new DeweyDecimal("1.2.0")) == 0);
     }
 
-    // XXX isGreaterThan, ...
+    // TODO isGreaterThan, ...
 
 }



Mime
View raw message