[ 
http://issues.apache.org/jira/browse/HARMONY-21?page=comments#action_12359731 ] 

Vladimir Strigun commented on HARMONY-21:
-----------------------------------------

I've found that my fix isn't good for Linux. When I try to create file using 
File("//usr") I get incorrect absolute path: "//usr" instead of "/usr".
To fix this additional check of of separatorChar should be added.
Here is updated fix:
254c254 
< if ((foundSlash && i == uncIndex) || !foundSlash) { 
--- 
> if ((foundSlash && i == uncIndex && path.charAt(0) == '/' && pathChar != '\\' 
>  && separatorChar == '\\') || !foundSlash) { 


> File constructor produces wrong path to absolute path specified relative to 
> root
> --------------------------------------------------------------------------------
>
>          Key: HARMONY-21
>          URL: http://issues.apache.org/jira/browse/HARMONY-21
>      Project: Harmony
>         Type: Bug
>   Components: Classlib
>     Reporter: Tim Ellison
>     Assignee: Geir Magnusson Jr

>
> Specifying an absolute path relative to the root directory produces the wrong 
> final pathname.
> Here's a simple test illustrating the problem:
>       public void test_File_Constructor() {
>               File path = new File("/dir/file");
>               File root = new File("/");
>               File file = new File(root, "/dir/file");
>               assertEquals("wrong path result ", path.getPath(), 
> file.getPath());
>       }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to