samepath: use java"

Author: scivision

+stdlib/+fileio/samepath.m

@@ -5,6 +5,24 @@
   path2 string {mustBeScalarOrEmpty}
 end
 
-issame = stdlib.fileio.absolute_path(path1) == stdlib.fileio.absolute_path(path2);
+if isempty(path1) || isempty(path2)
+   issame = logical.empty;
+   return
+end
+
+% absolute_path because unlike c++ filesystem, isSameFile does not
+% canonicalize first
+p1 = java.io.File(stdlib.fileio.absolute_path(path1)).toPath();
+p2 = java.io.File(stdlib.fileio.absolute_path(path2)).toPath();
+
+try
+  issame = java.nio.file.Files.isSameFile(p1, p2);
+catch e
+  if e.identifier == "MATLAB:Java:GenericException" && contains(e.message, "NoSuchFileException")
+    issame = false;
+  else
+    rethrow(e)
+  end
+end
 
 end

+stdlib/+test/TestFileImpure.m

@@ -15,12 +15,26 @@ function test_expanduser(tc)
 tc.verifyTrue(all(endsWith(expanduser(["~/abc", "~/123"]), ["abc", "123"])))
 end
 
+
 function test_makedir(tc)
 d = tempname;
 stdlib.makedir(d)
 tc.assertTrue(isfolder(d))
 end
 
+
+function test_samepath(tc)
+import stdlib.fileio.samepath
+tc.assumeTrue(usejava("jvm"), "Java required for samepath")
+
+tc.verifyEmpty(samepath(string.empty, string.empty))
+tc.verifyTrue(samepath("", ""))
+tc.verifyFalse(samepath(tempname, tempname))
+tc.verifyTrue(samepath("~/b/..", "~/c/.."), "tilde path ..")
+tc.verifyTrue(samepath(".", "a/.."))
+end
+
+
 function test_which_name(tc)
 
 tc.verifyEmpty(stdlib.which(tempname))

+stdlib/+test/TestFilePure.m

@@ -13,16 +13,6 @@ function test_posix(tc)
 end
 end
 
-function test_samepath(tc)
-tc.assumeTrue(usejava("jvm"), "Java required for samepath")
-
-tc.verifyEmpty(stdlib.samepath(string.empty, string.empty))
-tc.verifyTrue(stdlib.samepath("", ""))
-tc.verifyFalse(stdlib.samepath("a", "b"))
-tc.verifyTrue(stdlib.samepath("a/b/..", "a/c/.."))
-tc.verifyTrue(stdlib.samepath(".", "a/.."))
-
-end
 
 function test_path_tail(tc)