Converted int literals to size_t (#21840)

goedderz · web-flow · commit 839bb14e79c8 · 2025-07-07T11:56:00.000+02:00
diff --git a/arangod/RestServer/ApiRecordingFeature.cpp b/arangod/RestServer/ApiRecordingFeature.cpp
@@ -72,8 +72,10 @@ void ApiRecordingFeature::collectOptions(
   options->addOption(
       "--server.api-recording-memory-limit",
       "Size limit for the list of API call records.",
-      new UInt64Parameter(&_totalMemoryLimitCalls, 1, 256 * 1024,
-                          256 * 1024 * 1024 * 1024),
+      new UInt64Parameter(&_totalMemoryLimitCalls, 1,
+                          256 * (std::size_t{1} << 10),  // Min: 256 KiB
+                          256 * (std::size_t{1} << 30)   // Max: 256 GiB
+                          ),
       arangodb::options::makeDefaultFlags(arangodb::options::Flags::Uncommon));
 
   options->addOption(
@@ -85,8 +87,10 @@ void ApiRecordingFeature::collectOptions(
   options->addOption(
       "--server.aql-recording-memory-limit",
       "Size limit for the list of AQL query records.",
-      new UInt64Parameter(&_totalMemoryLimitQueries, 1, 256 * 1024,
-                          256 * 1024 * 1024 * 1024),
+      new UInt64Parameter(&_totalMemoryLimitQueries, 1,
+                          256 * (std::size_t{1} << 10),  // Min: 256 KiB
+                          256 * (std::size_t{1} << 30)   // Max: 256 GiB
+                          ),
       arangodb::options::makeDefaultFlags(arangodb::options::Flags::Uncommon));
 
   options
diff --git a/arangod/RestServer/ApiRecordingFeature.h b/arangod/RestServer/ApiRecordingFeature.h
@@ -159,10 +159,11 @@ class ApiRecordingFeature : public ArangodFeature {
   bool _enabledQueries{true};
 
   // Total memory limit for all ApiCallRecord lists combined
-  size_t _totalMemoryLimitCalls{25 * 1024 * 1024};  // Default: ~25MiB
+  size_t _totalMemoryLimitCalls{25 * (std::size_t{1} << 20)};  // Default: 25MiB
 
   // Total memory limit for all AqlCallRecord lists combined
-  size_t _totalMemoryLimitQueries{25 * 1024 * 1024};  // Default: ~25MiB
+  size_t _totalMemoryLimitQueries{25 *
+                                  (std::size_t{1} << 20)};  // Default: 25MiB
 
   // Memory limit for one list of ApiCallRecords (calculated as
   // _totalMemoryLimitCalls / NUMBER_OF_API_RECORD_LISTS)
