You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by cu...@apache.org on 2001/08/09 13:57:33 UTC
cvs commit: xml-xalan/test/java/src/org/apache/qetest QetestUtils.java
curcuru 01/08/09 04:57:33
Modified: test/java/src/org/apache/qetest QetestUtils.java
Log:
Fix possible bug: when checking for leading slash, check the current tmp
name, not the original filename, so we always end up with 3 slashes;
this should result in better names on UNIX systems
Revision Changes Path
1.8 +5 -5 xml-xalan/test/java/src/org/apache/qetest/QetestUtils.java
Index: QetestUtils.java
===================================================================
RCS file: /home/cvs/xml-xalan/test/java/src/org/apache/qetest/QetestUtils.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- QetestUtils.java 2001/06/28 13:17:04 1.7
+++ QetestUtils.java 2001/08/09 11:57:32 1.8
@@ -68,7 +68,7 @@
* line with a full or partial classname (in the org.apache.qetest
* area, obviously) and we'll load and execute that class instead.
* @author shane_curcuru@lotus.com
- * @version $Id: QetestUtils.java,v 1.7 2001/06/28 13:17:04 curcuru Exp $
+ * @version $Id: QetestUtils.java,v 1.8 2001/08/09 11:57:32 curcuru Exp $
*/
public abstract class QetestUtils
{
@@ -130,10 +130,10 @@
tmp = tmp.replace('\\', '/');
}
// Note the presumption that it's a file reference
- // Attempt to not add too many extra slashes on the
- // front if it already starts with a slash
- //@todo evaluate if this is really correct!
- if (filename.startsWith("/"))
+ // Ensure we have the correct number of slashes at the
+ // start: we always want 3 /// if it's absolute
+ // (which we should have forced above)
+ if (tmp.startsWith("/"))
return "file://" + tmp;
else
return "file:///" + tmp;
---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-cvs-help@xml.apache.org