canonical: use matlab.io to simplify

Author: scivision

+stdlib/canonical.m

@@ -17,22 +17,40 @@
 end
 
 if strempty(p)
-  c = '';
-else
-  [s, r] = fileattrib(p);
+  c = "";
+  return
+end
 
-  if s == 1
-    c = stdlib.posix(r.Name);
-  else
+if stdlib.isoctave(p) || isMATLABReleaseOlderThan('R2024a')
+  c = acanon(p);
+else
+  pth = matlab.io.internal.filesystem.resolvePath(p);
+  c = pth.ResolvedPath;
+  if strempty(c)
     c = stdlib.normalize(p);
   end
 end
 
-if isstring(p)
+try %#ok<*TRYNC>
   c = string(c);
 end
 
 end
 
+
+function c = acanon(p)
+
+if strempty(p), c = ''; return, end
+
+[s, r] = fileattrib(p);
+
+if s == 1
+  c = r.Name;
+else
+  c = stdlib.normalize(p);
+end
+
+end
+
 %!assert(canonical(""), "")
 %!assert(canonical("."), pwd())

test/TestCanonical.m

@@ -1,11 +1,11 @@
 classdef TestCanonical < matlab.unittest.TestCase
 
 properties(TestParameter)
-p = {{'', ''}, ...
+p = {{'', ""}, ...
 {"", ""}, ...
 {"not-exist", "not-exist"}, ...
 {"a/../b", "b"}, ...
-{strcat(mfilename("fullpath"), '.m/..'), stdlib.parent(mfilename("fullpath"))}, ...
+{strcat(mfilename("fullpath"), '.m/..'), string(fullfile(stdlib.parent(mfilename("fullpath"))))}, ...
 {"not-exist/a/..", "not-exist"}, ...
 {"./not-exist", "not-exist"}, ...
 {"../not-exist", "../not-exist"}