1. Change variable names in getPropertyGroups

2. Use strings instead of cellstrs/chars
mathworks · Jul 26, 2023 · b5d1c2a · b5d1c2a
1 parent 438e97f
commit b5d1c2a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/matlab/src/matlab/+arrow/+type/TimestampType.m b/matlab/src/matlab/+arrow/+type/TimestampType.m
@@ -41,9 +41,9 @@
     end
 
     methods (Access=protected)
-        function propgrp = getPropertyGroups(~)
-          proplist = {'ID', 'TimeUnit', 'TimeZone'};
-          propgrp = matlab.mixin.util.PropertyGroup(proplist);
+        function group = getPropertyGroups(~)
+          targets = ["ID" "TimeUnit" "TimeZone"];
+          group = matlab.mixin.util.PropertyGroup(targets);
         end
     end
 end
diff --git a/matlab/test/arrow/type/tTimestampType.m b/matlab/test/arrow/type/tTimestampType.m
@@ -118,7 +118,7 @@ function Display(testCase)
         %    TimeZone: "America/Anchorage"
         %
             type = arrow.timestamp(TimeUnit="Second", TimeZone="America/Anchorage"); %#ok<NASGU>
-            classnameLink = '<a href="matlab:helpPopup arrow.type.TimestampType" style="font-weight:bold">TimestampType</a>';
+            classnameLink = "<a href=""matlab:helpPopup arrow.type.TimestampType"" style=""font-weight:bold"">TimestampType</a>";
             header = "  " + classnameLink + " with properties:" + newline;
             body = strjust(pad(["ID:"; "TimeUnit:"; "TimeZone:"]));
             body = body + " " + ["Timestamp"; "Second"; """America/Anchorage"""];