test, backend: more efficient, robust

Author: scivision

+stdlib/+dotnet/create_symlink.m

@@ -9,11 +9,9 @@
 end
 
 % https://learn.microsoft.com/en-us/dotnet/api/system.io.file.createsymboliclink
-try
+try %#ok<TRYNC>
   System.IO.File.CreateSymbolicLink(link, target);
   ok = true;
-catch e
-  warning(e.identifier, "%s", e.message)
 end
 
 end

+stdlib/+dotnet/read_symlink.m

@@ -1,17 +1,16 @@
 %% DOTNET.READ_SYMLINK resolve the symbolic links of a filepath
 
-function r = read_symlink(p)
+function r = read_symlink(file)
 
 try
-  h = System.IO.FileInfo(p);
+  h = System.IO.FileInfo(file);
   r = string(h.LinkTarget);
   % on Unix, this can be empty if the file is not a symlink
+  if isempty(r)
+    r = "";
+  end
 catch
   r = "";
 end
 
-if isempty(r)
-  r = "";
-end
-
 end

+stdlib/+java/device.m

@@ -6,7 +6,11 @@
 % Java 1.8 benefits from the absolute() for stability
 % seen on older Matlab versions on HPC
 
-i = java.nio.file.Files.getAttribute(jp, 'unix:dev', opt);
+try
+  i = java.nio.file.Files.getAttribute(jp, 'unix:dev', opt);
+catch
+  i = [];
+end
 
 i = uint64(i);
 end

+stdlib/+java/hard_link_count.m

@@ -1,6 +1,9 @@
 function c = hard_link_count(p)
 
 opt = java.nio.file.LinkOption.values();
-c = java.nio.file.Files.getAttribute(javaPathObject(p), "unix:nlink", opt);
-
-end
+try
+  c = java.nio.file.Files.getAttribute(javaPathObject(p), "unix:nlink", opt);
+catch
+  c = [];
+end
+end

+stdlib/+java/inode.m

@@ -6,7 +6,11 @@
 % Java 1.8 benefits from the absolute() for stability--it's not an issue
 % on every computer.
 
-i = java.nio.file.Files.getAttribute(jp, "unix:ino", opt);
+try
+  i = java.nio.file.Files.getAttribute(jp, "unix:ino", opt);
+catch
+  i = [];
+end
 
 i = uint64(i);
 end

+stdlib/+python/cpu_load.m

@@ -7,6 +7,4 @@
   L = double(L(1));
 end
 
-
-
 end

+stdlib/+python/filesystem_type.m

@@ -2,22 +2,21 @@
 
 t = '';
 
+% important for heuristic matching
 if ~stdlib.exists(p)
   return
 end
 
 pr = stdlib.absolute(p);
 
-try
+try %#ok<TRYNC>
   for part = py.psutil.disk_partitions(p)
     prt = part{1};
     if startsWith(pr, char(prt.mountpoint))
       t = char(prt.fstype);
       return
     end
   end
-catch e
-  warning(e.identifier, "filesystem_type(%s) failed: %s", p, e.message);
 end
 
 end

+stdlib/+python/get_owner.m

@@ -1,8 +1,8 @@
 function n = get_owner(p)
 
-if stdlib.exists(p)
+try
   n = string(py.str(py.pathlib.Path(p).owner()));
-else
+catch
   n = "";
 end
 

+stdlib/+sys/samepath.m

@@ -1,13 +1,18 @@
 function y = samepath(path1, path2)
 
-assert(~ispc(), "Unix-like only")
-
 y = false;
 
 if ~stdlib.exists(path1) || ~stdlib.exists(path2)
   return
 end
 
+if ispc()
+  y = stdlib.sys.device(path1) == stdlib.sys.device(path2) && ...
+      stdlib.sys.inode(path1) == stdlib.sys.inode(path2);
+  return
+end
+
+
 if ismac()
   flag = '-f';
 else

+stdlib/canonical.m

@@ -21,7 +21,7 @@
   backend (1,:) string = ["native", "legacy"]
 end
 
-[fun, b] = hbackend(backend, "canonical", 'R2024a');
+[fun, b] = hbackend(backend, "canonical");
 
 if isscalar(p) || b == "native"
   c = fun(p, strict);