simplify get_max_open_Files()

Author: scivision

+stdlib/+python/get_max_open_files.m

@@ -1,22 +1,19 @@
 %% python.get_max_open_files  Get process open-file soft limit via Python
 function omax = get_max_open_files()
 
-omax = [];
+omax = missing;
 
 if ispc()
-	omax = uint64(omax);
-	return
+  return
 end
 
 try
 	% RLIMIT_NOFILE soft limit is the active per-process descriptor cap.
 	lim = py.resource.getrlimit(py.resource.RLIMIT_NOFILE);
-	omax = lim(1);
+	omax = uint64(lim(1));
 catch e
 	pythonException(e)
-	omax = [];
+	omax = missing;
 end
 
-omax = uint64(omax);
-
 end

+stdlib/+shell/get_max_open_files.m

@@ -1,21 +1,19 @@
 %% shell.get_max_open_files  Get process open-file soft limit via shell
 function omax = get_max_open_files()
 
-omax = [];
+omax = missing;
 
 if ~ispc()
 
 [s, m] = system('ulimit -n');
 if s == 0
-  omax = str2double(strtrim(m));
+  o = str2double(strtrim(m));
 
-  if ~isfinite(omax) || omax <= 0
-    omax = [];
+  if isfinite(o) && o > 0
+    omax = uint64(o);
   end
 end
 
 end
 
-omax = uint64(omax);
-
 end

+stdlib/get_max_open_files.m

@@ -7,7 +7,7 @@
   backend (1,:) string = ["python", "shell"]
 end
 
-omax = uint64([]);
+omax = missing;
 
 for b = backend
 	switch b
@@ -21,7 +21,7 @@
 			error('stdlib:get_max_open_files:ValueError', 'Unknown backend: %s', b)
 	end
 
-	if ~isempty(omax)
+	if ~ismissing(omax)
 		return
 	end
 end

test/TestDisk.m

@@ -62,10 +62,10 @@ function test_disk_capacity(tc, Ps, B_jdps)
 function test_ulimit(tc, B_ps)
 [i, b] = stdlib.get_max_open_files(B_ps);
 tc.assertMatches(b, B_ps)
-tc.verifyClass(i, 'uint64')
 if ispc() || (B_ps == "python" && ~stdlib.has_python())
-  tc.verifyEmpty(i)
+  tc.verifyEqual(i, missing)
 else
+  tc.verifyClass(i, 'uint64')
   tc.verifyGreaterThan(i, 0)
 end
 end