symlink use missing

scivision · scivision · commit 80917145088a · 2026-06-23T11:19:17.000-05:00
set_modtime use missing
diff --git a/+stdlib/+dotnet/create_symlink.m b/+stdlib/+dotnet/create_symlink.m
@@ -13,7 +13,7 @@
   ok = true;
 catch e
   dotnetException(e)
-  ok = logical([]);
+  ok = missing;
 end
 
 end
diff --git a/+stdlib/+dotnet/is_symlink.m b/+stdlib/+dotnet/is_symlink.m
@@ -18,7 +18,7 @@
   end
 catch e
   dotnetException(e)
-  y = logical([]);
+  y = missing;
 end
 
 end
diff --git a/+stdlib/+java/is_symlink.m b/+stdlib/+java/is_symlink.m
@@ -10,7 +10,7 @@
   y = javaMethod('isSymbolicLink', 'java.nio.file.Files', p);
 catch e
   javaException(e)
-  y = logical([]);
+  y = missing;
 end
 
 end
diff --git a/+stdlib/+java/set_modtime.m b/+stdlib/+java/set_modtime.m
@@ -16,7 +16,7 @@
   ok = javaMethod('setLastModified', o, int64(utc) * 1000);
 catch e
   javaException(e)
-  ok = logical([]);
+  ok = missing;
 end
 
 end
diff --git a/+stdlib/+native/create_symlink.m b/+stdlib/+native/create_symlink.m
@@ -1,5 +1,11 @@
 function i = create_symlink(target, link)
 
+i = missing;
+
+if stdlib.matlabOlderThan('R2024b')
+  return
+end
+
 try
   createSymbolicLink(link, target);
   i = true;
@@ -12,8 +18,6 @@
       else
         rethrow(e)
       end
-    case {'MATLAB:UndefinedFunction'}
-      i = logical([]);
     case {'MATLAB:io:filesystem:symlink:FileExists', 'MATLAB:io:filesystem:symlink:TargetNotFound'}
       i = false;
     otherwise
diff --git a/+stdlib/+native/is_symlink.m b/+stdlib/+native/is_symlink.m
@@ -3,7 +3,7 @@
 if ~stdlib.matlabOlderThan('R2024b')
   i = isSymbolicLink(file);
 else
-  i = logical([]);
+  i = missing;
 end
 
 end
diff --git a/+stdlib/+python/create_symlink.m b/+stdlib/+python/create_symlink.m
@@ -14,7 +14,7 @@
   ok = true;
 catch e
   pythonException(e)
-  ok = logical([]);
+  ok = missing;
 end
 
 end
diff --git a/+stdlib/+python/is_symlink.m b/+stdlib/+python/is_symlink.m
@@ -9,7 +9,7 @@
   y = py.pathlib.Path(file).is_symlink();
 catch e
   pythonException(e)
-  y = logical([]);
+  y = missing;
 end
 
 end
diff --git a/+stdlib/+python/set_modtime.m b/+stdlib/+python/set_modtime.m
@@ -10,7 +10,7 @@
   if e.identifier == "MATLAB:Python:PyException" && contains(e.message, "FileNotFoundError")
     ok = false;
   else
-    ok = logical([]);
+    ok = missing;
   end
 end
 
diff --git a/+stdlib/create_symlink.m b/+stdlib/create_symlink.m
@@ -14,7 +14,7 @@
   backend (1,:) string = ["native", "dotnet", "python", "shell"]
 end
 
-i = logical([]);
+i = missing;
 
 for b = backend
   switch b
@@ -32,7 +32,7 @@
       error('stdlib:create_symlink:ValueError', 'Unknown backend: %s', b)
   end
 
-  if ~isempty(i)
+  if ~ismissing(i)
     return
   end
 end
diff --git a/+stdlib/is_symlink.m b/+stdlib/is_symlink.m
@@ -14,7 +14,7 @@
 end
 
 
-i = logical([]);
+i = missing;
 
 for b = backend
   switch b
@@ -34,7 +34,7 @@
       error('stdlib:is_symlink:ValueError', 'Unknown backend: %s', b)
   end
 
-  if ~isempty(i)
+  if ~ismissing(i)
     return
   end
 end
diff --git a/+stdlib/set_modtime.m b/+stdlib/set_modtime.m
@@ -15,7 +15,7 @@
   backend (1,:) string = ["java", "python", "shell"]
 end
 
-i = logical([]);
+i = missing;
 
 for b = backend
   switch b
@@ -31,7 +31,7 @@
       error('stdlib:set_modtime:ValueError', 'Unknown backend: %s', b)
   end
 
-  if ~isempty(i)
+  if ~ismissing(i)
     return
   end
 end
diff --git a/example/BenchmarkSymlink.m b/example/BenchmarkSymlink.m
@@ -51,7 +51,6 @@ function bench_is_symlink_not_exist(tc, is_backend)
 tc.stopMeasuring()
 
 tc.verifyClass(i, 'logical')
-tc.assertNotEmpty(i)
 tc.verifyFalse(i)
 end
 
diff --git a/test/TestBackend.m b/test/TestBackend.m
@@ -144,7 +144,7 @@ function test_backend(tc)
 tc.assertTrue(stdlib.touch('time.txt'))
 [i, b] = stdlib.set_modtime("time.txt", datetime());
 tc.assertThat(b, IsSubsetOf(stdlib.Backend('set_modtime').backends))
-tc.verifyNotEmpty(i)
+tc.verifyTrue(i)
 
 
 end
diff --git a/test/TestSymlink.m b/test/TestSymlink.m
@@ -25,7 +25,8 @@ function setup_symlink(tc)
 tc.target = fullfile(pwd(), 'my_target.txt');
 tc.assertTrue(stdlib.touch(tc.target), 'failed to create test target')
 
-tc.assertTrue(stdlib.create_symlink(tc.target, tc.link), 'failed to create test link')
+ok = stdlib.create_symlink(tc.target, tc.link);
+tc.assertTrue(ok, 'failed to create test link')
 end
 end
 
@@ -43,7 +44,7 @@ function test_is_symlink(tc, B_is_symlink)
   tc.verifyFalse(stdlib.is_symlink('', B_is_symlink))
   tc.verifyFalse(stdlib.is_symlink(tc.target, B_is_symlink))
 else
-  tc.verifyEmpty(i)
+  tc.verifyEqual(i, missing)
 end
 end
 
@@ -77,9 +78,9 @@ function test_create_symlink(tc, B_create_symlink)
 if ismember(B_create_symlink, stdlib.Backend().select('create_symlink'))
   tc.verifyTrue(r)
 elseif ispc() && strcmp(B_create_symlink, 'native')
-  tc.verifyTrue(isempty(r) || r)
+  tc.verifyTrue(ismissing(r) || r)
 else
-  tc.verifyEmpty(r)
+  tc.verifyEqual(r, missing)
 end
 end
 
@@ -90,12 +91,12 @@ function test_create_symlink_empty(tc, B_create_symlink)
 
 if ismember(B_create_symlink, stdlib.Backend().select('create_symlink'))
   if ispc() && strcmp(B_create_symlink, 'native')
-    tc.verifyTrue( isempty(i) || ~i )
+    tc.verifyTrue( ismissing(i) || ~i )
   else
     tc.verifyFalse(i)
   end
 else
-  tc.verifyTrue( isempty(i) || ~i )
+  tc.verifyTrue( ismissing(i) || ~i )
 end
 end
 
@@ -105,12 +106,12 @@ function test_create_symlink_exist(tc, B_create_symlink)
 
 if ismember(B_create_symlink, stdlib.Backend().select('create_symlink'))
   if ispc() && strcmp(B_create_symlink, 'native')
-    tc.verifyTrue( isempty(i) || ~i )
+    tc.verifyTrue( ismissing(i) || ~i )
   else
     tc.verifyFalse(i, 'should fail for existing symlink')
   end
 else
-  tc.verifyTrue( isempty(i) || ~i )
+  tc.verifyTrue( ismissing(i) || ~i )
 end
 
 end
diff --git a/test/TestTime.m b/test/TestTime.m
@@ -36,7 +36,7 @@ function test_touch_modtime(tc, B_jps)
   t1 = stdlib.get_modtime(fn);
   tc.verifyGreaterThanOrEqual(t1, t0)
 else
-  tc.assertEmpty(ok)
+  tc.assertEqual(ok, missing)
 end
 end
 
