peterreilly 2004/12/13 10:52:17 Modified: . Tag: ANT_16_BRANCH WHATSNEW src/main/org/apache/tools/ant/taskdefs/optional Tag: ANT_16_BRANCH XMLValidateTask.java Log: sync Revision Changes Path No revision No revision 1.503.2.148 +3 -0 ant/WHATSNEW Index: WHATSNEW =================================================================== RCS file: /home/cvs/ant/WHATSNEW,v retrieving revision 1.503.2.147 retrieving revision 1.503.2.148 diff -u -r1.503.2.147 -r1.503.2.148 --- WHATSNEW 10 Dec 2004 15:20:10 -0000 1.503.2.147 +++ WHATSNEW 13 Dec 2004 18:52:17 -0000 1.503.2.148 @@ -79,6 +79,9 @@ twice - if the resource is in the project classpath and if the classloader is requested with a null path. +* XMLValidate used URL#getFile rather than the ant method FileUtils#fromURI + Bugzilla report 32508 + Changes from Ant 1.6.1 to Ant 1.6.2 =================================== No revision No revision 1.36.2.6 +17 -15 ant/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java Index: XMLValidateTask.java =================================================================== RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java,v retrieving revision 1.36.2.5 retrieving revision 1.36.2.6 diff -u -r1.36.2.5 -r1.36.2.6 --- XMLValidateTask.java 2 Jun 2004 21:13:18 -0000 1.36.2.5 +++ XMLValidateTask.java 13 Dec 2004 18:52:17 -0000 1.36.2.6 @@ -19,8 +19,6 @@ import java.io.File; import java.io.FileInputStream; import java.io.IOException; -import java.net.MalformedURLException; -import java.net.URL; import java.util.Vector; import org.apache.tools.ant.AntClassLoader; @@ -59,7 +57,7 @@ /** * helper for path -> URI and URI -> path conversions. */ - private static FileUtils fu = FileUtils.newFileUtils(); + private static final FileUtils FILE_UTILS = FileUtils.newFileUtils(); protected static final String INIT_FAILED_MSG = "Could not start xml validation: "; @@ -455,7 +453,7 @@ log("Validating " + afile.getName() + "... ", Project.MSG_VERBOSE); errorHandler.init(afile); InputSource is = new InputSource(new FileInputStream(afile)); - String uri = fu.toURI(afile.getAbsolutePath()); + String uri = FILE_UTILS.toURI(afile.getAbsolutePath()); is.setSystemId(uri); xmlReader.parse(is); } catch (SAXException ex) { @@ -545,18 +543,22 @@ private String getMessage(SAXParseException e) { String sysID = e.getSystemId(); if (sysID != null) { - try { - int line = e.getLineNumber(); - int col = e.getColumnNumber(); - return new URL(sysID).getFile() - + (line == -1 - ? "" - : (":" + line + (col == -1 ? "" : (":" + col)))) - + ": " - + e.getMessage(); - } catch (MalformedURLException mfue) { - // ignore and just return exception message + String name = sysID; + if (sysID.startsWith("file:")) { + try { + name = FILE_UTILS.fromURI(sysID); + } catch (Exception ex) { + // if this is not a valid file: just use the uri + } } + int line = e.getLineNumber(); + int col = e.getColumnNumber(); + return name + + (line == -1 + ? "" + : (":" + line + (col == -1 ? "" : (":" + col)))) + + ": " + + e.getMessage(); } return e.getMessage(); }
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]