> Index: MakeUrl.java
> ===================================================================
> ...
> * @ant.task category="core" name="tourl"
s/tourl/makeurl/ you are using that in the default.properties (not the
SmartFrog name :-)
/**
* error message
*/
public static final String ERROR_MISSING_FILE = "A source file is missing
:";
public static final String ERROR_NO_PROPERTY = "No property defined";
public static final String ERROR_NO_FILES = "No files defined";
A JavaDoc comment only for the first? No [EMAIL PROTECTED] ? <g>
private String filesetsToURL() {
...
log(asUrl, Project.MSG_DEBUG);
Would the output of the origin file be helpful?
log(f + " --> " + asUrl, Project.MSG_DEBUG);
Mmh - same log statement in pathsToURL() ...
public void execute() throws BuildException {
validate();
//now exit here if the property is already set
if (getProject().getProperty(property) != null) {
return;
}
A log statement would be helpful
if (getProject().getProperty(property) != null) {
log("property '" + property + "' already set, exiting <makeurl>",
Project.MSG_VERBOSE);
//same log level as setting the property
return;
}
cheers
Jan
>-----Ursprüngliche Nachricht-----
>Von: Stefan Bodewig [mailto:[EMAIL PROTECTED]
>Gesendet: Dienstag, 23. August 2005 06:06
>An: [email protected]
>Betreff: Re: cvs commit:
>ant/src/testcases/org/apache/tools/ant/taskdefs MakeUrlTest.java
>
>On 22 Aug 2005, <[EMAIL PROTECTED]> wrote:
>
>> url = fileToConvert.toURI().toURL().toExternalForm();
>
>I think File.toURI is JDK 1.4+.
>
>What is wrong with FileUtils.toURL()? Shouldn't we improve
>FileUtils and use that from the task if anything is wrong?
>
>Stefan
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: [EMAIL PROTECTED] For
>additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
