inode, device: 20x faster

Author: scivision

+stdlib/+java/device.m

@@ -8,9 +8,9 @@
 
 % Java 1.8 benefits from the absolute() for stability
 % seen on older Matlab versions on HPC
-opt = java.nio.file.LinkOption.values();
 
 try
+  opt = java.nio.file.LinkOption.values();
   i = java.nio.file.Files.getAttribute(javaAbsolutePath(file), "unix:dev", opt);
 catch e
   javaException(e)

+stdlib/+java/inode.m

@@ -8,9 +8,9 @@
 
 % Java 1.8 benefits from the absolute() for stability
 % seen on older Matlab versions on HPC
-opt = java.nio.file.LinkOption.values();
 
 try
+  opt = java.nio.file.LinkOption.values();
   i = java.nio.file.Files.getAttribute(javaAbsolutePath(file), "unix:ino", opt);
 catch e
   javaException(e)

+stdlib/+java/private/javaException.m

@@ -1,8 +1,17 @@
 function javaException(e)
 
-switch class(e.ExceptionObject)
-  case {'java.nio.file.NoSuchFileException', 'java.nio.file.NotLinkException'}
-  otherwise, rethrow(e)
+switch e.identifier
+  case 'MATLAB:Java:GenericException'
+    switch class(e.ExceptionObject)
+      case {'java.nio.file.NoSuchFileException', 'java.nio.file.NotLinkException', 'java.lang.UnsupportedOperationException'}
+        % pass
+      otherwise
+        rethrow(e)
+    end
+  case 'MATLAB:undefinedVarOrClass'
+    % Java not enabled -nojvm
+  otherwise
+    rethrow(e)
 end
 
 end

+stdlib/device.m

@@ -2,18 +2,35 @@
 %
 %%% Inputs
 % * file: path to file
+% * backend: backend to use
 %%% Outputs
 % * i: device index
 % * b: backend used
 
 function [i, b] = device(file, backend)
 arguments
   file string
-  backend (1,:) string = ["java", "python", "perl", "sys"]
+  backend (1,:) string = ["java", "python", "sys"]
 end
 
-o = stdlib.Backend(mfilename(), backend);
-i = o.func(file);
+i = uint64.empty;
+
+for b = backend
+  switch b
+    case "java"
+      i = stdlib.java.device(file);
+    case "python"
+      if stdlib.matlabOlderThan('R2022a'), continue, end
+      i = stdlib.python.device(file);
+    case "sys"
+      i = stdlib.sys.device(file);
+    otherwise
+      error("stdlib:device:ValueError", "Unknown backend: %s", b)
+  end
+
+  if ~isempty(i)
+    return
+  end
+end
 
-b = o.backend;
 end

+stdlib/hard_link_count.m

@@ -2,27 +2,39 @@
 %
 %%% inputs
 % * file: path to check
+% * backend: backend to use
 %%% Outputs
-% * c: number of hard links
+% * i: number of hard links
 % * b: backend used
 %% Java backend references:
 %
 % * https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/nio/file/Files.html#getPosixFileAttributes(java.nio.file.Path,java.nio.file.LinkOption...)
 % * https://docs.oracle.com/javase/tutorial/essential/io/links.html
 
-function [c, b] = hard_link_count(file, backend)
+function [i, b] = hard_link_count(file, backend)
 arguments
   file string
   backend (1,:) string = ["java", "python", "sys"]
 end
 
-o = stdlib.Backend(mfilename(), backend);
+i = [];
 
-if isscalar(file)
-  c = o.func(file);
-else
-  c = arrayfun(o.func, file);
+for b = backend
+  switch b
+    case "java"
+      i = stdlib.java.hard_link_count(file);
+    case "python"
+      if stdlib.matlabOlderThan('R2022a'), continue, end
+      i = stdlib.python.hard_link_count(file);
+    case "sys"
+      i = stdlib.sys.hard_link_count(file);
+    otherwise
+      error("stdlib:hard_link_count:ValueError", "Unknown backend: %s", b)
+  end
+
+  if ~isempty(i)
+    return
+  end
 end
 
-b = o.backend;
 end

+stdlib/inode.m

@@ -2,24 +2,35 @@
 %
 %%% inputs
 % * file: path to check
+% * backend: backend to use
 %%% Outputs
 % * i: inode number
 % * b: backend used
 
 function [i, b] = inode(file, backend)
 arguments
   file string
-  backend (1,:) string = ["java", "python", "perl", "sys"]
+  backend (1,:) string = ["java", "python", "sys"]
 end
 
-o = stdlib.Backend(mfilename(), backend);
+i = uint64.empty;
 
-if isscalar(file)
-  i = o.func(file);
-else
-  i = arrayfun(o.func, file);
-end
+for b = backend
+  switch b
+    case "java"
+      i = stdlib.java.inode(file);
+    case "python"
+      if stdlib.matlabOlderThan('R2022a'), continue, end
+      i = stdlib.python.inode(file);
+    case "sys"
+      i = stdlib.sys.inode(file);
+    otherwise
+      error("stdlib:inode:ValueError", "Unknown backend: %s", b)
+  end
 
-b = o.backend;
+  if ~isempty(i)
+    return
+  end
+end
 
 end

+stdlib/samepath.m

@@ -11,7 +11,7 @@
 %
 %%% inputs
 % * path1, path2: paths to compare
-% * backend: backend to use, default is "python"
+% * backend: backend to use
 %%% Outputs
 % * ok: true if paths are the same
 % * b: backend used

+stdlib/+perl/device.m renamed to example/+perl/device.m

@@ -1,3 +1,6 @@
+%% PERL.INODE
+% several times slower than stdlib.sys.inode
+
 function [r, cmd] = inode(file)
 
 r = uint64.empty;

+stdlib/+perl/inode.m renamed to example/+perl/inode.m

@@ -8,16 +8,9 @@
 end
 
 properties(TestParameter)
-backend
+backend = cellstr(["java", "python", "sys"])
 end
 
-methods (TestParameterDefinition, Static)
-function backend = setupBackend()
-  backend = init_backend('inode');
-end
-end
-
-
 methods (Test)
 
 function bench_exist(tc, backend)