ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From <Jan.Mate...@rzf.fin-nrw.de>
Subject AW: svn commit: r899468 - in /ant/antlibs/props/trunk: docs/index.html src/main/org/apache/ant/props/EncodeURLEvaluator.java src/main/org/apache/ant/props/antlib.xml src/tests/antunit/encodeURL-test.xml
Date Fri, 15 Jan 2010 06:53:47 GMT
Is this a candidate for FileUtils with a proper javadoc?

Jan 

>-----Urspr√ľngliche Nachricht-----
>Von: Matt Benson [mailto:gudnabrsam@gmail.com] 
>Gesendet: Freitag, 15. Januar 2010 00:52
>An: Ant Developers List
>Betreff: Re: svn commit: r899468 - in 
>/ant/antlibs/props/trunk: docs/index.html 
>src/main/org/apache/ant/props/EncodeURLEvaluator.java 
>src/main/org/apache/ant/props/antlib.xml 
>src/tests/antunit/encodeURL-test.xml
>
>To confirm that re-testing on OSX Tiger w/ Java 5 blows up when call  
>toURI() against a URL with a space included.
>
>On Jan 14, 2010, at 5:45 PM, Matt Benson wrote:
>
>> Yes--my testing seems to indicate that the single-arg constructor  
>> of URI is used in this case, which blows up for improperly escaped  
>> characters per its javadoc.  I'll double-check though.
>>
>> Thanks,
>> Matt
>>
>> On Jan 14, 2010, at 5:36 PM, Jesse Glick wrote:
>>
>>> mbenson@apache.org wrote:
>>>> URL: http://svn.apache.org/viewvc/ant/antlibs/props/trunk/src/ 
>>>> main/org/apache/ant/props/EncodeURLEvaluator.java? 
>>>> rev=899468&view=auto
>>>> 
>==================================================================== 
>>>> ==========
>>>> +            URI uri = new URI(url.getProtocol(), 
>url.getUserInfo(),
>>>> +                     url.getHost(), url.getPort(), url.getPath 
>>>> (), url.getQuery(), url.getRef());
>>>
>>> Something wrong with
>>>
>>>   URI uri = url.toURI();
>>>
>>> ?
>>>
>>> 
>---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
>>> For additional commands, e-mail: dev-help@ant.apache.org
>>>
>>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
>For additional commands, e-mail: dev-help@ant.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message