set_permissions simplify

Author: scivision

+stdlib/+legacy/set_permissions.m

@@ -34,7 +34,7 @@
 [s, msg, id] = fileattrib(char(file), mode);
 ok = s == 1;
 if ~ok
-  warning(id, 'Failed to set permissions %s for %s: %s', mode, file, msg)
+  error(id, 'Failed to set permissions %s for %s: %s', mode, file, msg)
 end
 
 end

+stdlib/+native/set_permissions.m

@@ -1,9 +1,5 @@
 function ok = set_permissions(file, readable, writable, executable)
 
-mustBeInteger(readable)
-mustBeInteger(writable)
-mustBeInteger(executable)
-
 p = filePermissions(file);
 
 assert(isscalar(p), "set_permissions: one file only")

+stdlib/set_permissions.m

@@ -13,26 +13,18 @@
 
 function [ok, b] = set_permissions(file, readable, writable, executable)
 arguments
-  file {mustBeTextScalar, mustBeFileOrFolder}
+  file {mustBeTextScalar}
   readable (1,1) {mustBeInteger, mustBeMember(readable, [-1, 0, 1])}
   writable (1,1) {mustBeInteger, mustBeMember(writable, [-1, 0, 1])}
   executable (1,1) {mustBeInteger, mustBeMember(executable, [-1, 0, 1])}
 end
 
-try
+if stdlib.matlabOlderThan('R2025a')
+  ok = stdlib.legacy.set_permissions(file, readable, writable, executable);
+  b = 'legacy';
+else
   ok = stdlib.native.set_permissions(file, readable, writable, executable);
   b = 'native';
-catch e
-  switch e.identifier
-    case {'MATLAB:UndefinedFunction', 'MATLAB:m_illegal_character'}
-      ok = stdlib.legacy.set_permissions(file, readable, writable, executable);
-      b = 'legacy';
-    case 'MATLAB:io:filesystem:filePermissions:CannotFindLocation'
-      ok = false;
-      b = '';
-    otherwise
-      rethrow(e)
-  end
 end
 
 end

test/TestPermissions.m

@@ -26,6 +26,11 @@ function test_not_exist(tc, Pe)
 e = 'MATLAB:io:filesystem:filePermissions:CannotFindLocation';
 tc.verifyError(@() stdlib.get_permissions(Pe), e)
 
+if stdlib.matlabOlderThan('R2025a')
+  e = 'MATLAB:FILEATTRIB:CannotFindFile';
+end
+tc.verifyError(@() stdlib.set_permissions(Pe, 0, 0, 0), e)
+
 end
 
 function test_get_permissions(tc, Ps)