[ 
https://issues.apache.org/jira/browse/XBEAN-109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12639377#action_12639377
 ] 

Benjamin Bentmann commented on XBEAN-109:
-----------------------------------------

+1 on the usage of {{File.toURI().toURL()}}

-1 on the replacement of {{new File(url.toURI())}} with {{new 
File(url.getPath())}}. {{URL.getPath()}} merely returns a substring of the URL 
and as such can still contain percent-encoded characters. Therefore, {{new 
File(url.getPath())}} should only be used as a fallback in case the 
URISyntaxException indicates an unescaped URL.

>  org.apache.xbean.classloader.JarFileClassLoader can not handle pathnames 
> with containing spaces
> ------------------------------------------------------------------------------------------------
>
>                 Key: XBEAN-109
>                 URL: https://issues.apache.org/jira/browse/XBEAN-109
>             Project: XBean
>          Issue Type: Bug
>          Components: classloader
>         Environment: jdk1.6, Windows 2000 Latest version from SVN
>            Reporter: Ingo Bormann
>         Attachments: xbean.diff
>
>
> A lot of classes in the package org.apache.xbean.classloader use File.toURL() 
> instead of File.toURI().toURL(). File.toURL() is deprecated and does not work 
> on windows with pathnames containing spaces. If a pathname contains spaces 
> then File.toURL() does not convert spaces correctly. Javadoc recommends to 
> use File.toURI().toURL() instead.
> I have a patched version where this is fixed for the full package 
> org.apache.xbean.classloader.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to