buildfile: use task

Author: scivision

buildfile.m

@@ -3,33 +3,46 @@
 
 plan.DefaultTasks = "test";
 
+if ~isMATLABReleaseOlderThan("R2023b")
+  plan("check") = matlab.buildtool.tasks.CodeIssuesTask("+stdlib", IncludeSubfolders=true);
+end
+
 if ~isMATLABReleaseOlderThan("R2024b")
 
   cxx = mex.getCompilerConfigurations('c++');
+  flags = cxx.Details.CompilerFlags;
+
+  msvc = startsWith(cxx.ShortName, "MSVCPP");
 
   % disp("Mex using: " + cxx.Name + " " + cxx.Version)
 
-  compiler_opt = "CXXFLAGS=$CXXFLAGS -std=c++17";
+  std = "-std=c++17";
+  compiler_id = "";
   % FIXME: Windows oneAPI
-  if startsWith(cxx.ShortName, "MSVCPP")
-    compiler_opt = "COMPFLAGS=$COMPFLAGS /std:c++17";
+  if msvc
+    std = "/std:c++17";
   elseif ismac
-    compiler_opt = "CXXFLAGS=$CXXFLAGS -std=c++17";
+    % keep for if-logic
   elseif isunix && cxx.ShortName == "g++"
-    compiler_opt = compiler_opt + " -fPIC";
     % FIXME: update when desired GCC != 10 for newer Matlab
     if isMATLABReleaseOlderThan("R2025b") && ~startsWith(cxx.Version, "10")
       % https://www.mathworks.com/help/matlab/matlab_external/choose-c-or-c-compilers.html
       % https://www.mathworks.com/help/matlab/matlab_external/change-default-gcc-compiler-on-linux-system.html
       [s, ~] = system("which g++-10");
       if s == 0
-        compiler_opt = "CXX=g++-10 " + compiler_opt;
+        compiler_id = "CXX=g++-10";
       else
         warning("GCC 10 not found, using default, may fail on runtime")
       end
     end
   end
 
+  if msvc
+    compiler_opt = "COMPFLAGS=" + flags + " " + std;
+  else
+    compiler_opt = compiler_id + " CXXFLAGS=" + flags + " " + std;
+  end
+
   root = plan.RootFolder;
   bin = fullfile(root, "bin");
   if ~isfolder(bin), mkdir(bin), end
@@ -46,17 +59,15 @@
 end
 
 
-function checkTask(~)
-% Identify code issues (recursively all Matlab .m files)
-issues = codeIssues("+stdlib");
-assert(isempty(issues.Issues), formattedDisplayText(issues.Issues))
-end
-
 function testTask(~)
   t = fileparts(mfilename("fullpath"));
   addpath(t)
 
-  r = runtests(IncludeSubfolders=true, strict=true, UseParallel=true, OutputDetail="Concise");
+  r = runtests("test", ...
+    IncludeSubfolders=true, ...
+    strict=true, ...
+    UseParallel=true, ...
+    OutputDetail="Concise");
 
   assert(~isempty(r), "No tests were run")
   assertSuccess(r)