]> git.ipfire.org Git - thirdparty/collectd.git/commitdiff
cpu plugin: Use a const instead of repeating a literal multiple times.
authorFlorian Forster <octo@collectd.org>
Fri, 19 Jan 2024 19:58:25 +0000 (20:58 +0100)
committerFlorian Forster <octo@collectd.org>
Mon, 22 Jan 2024 15:07:57 +0000 (16:07 +0100)
src/cpu_test.c

index ec4ed1b4f32506f48b4ef47564de532331625432..ac5c4bf4695ae5975cef9858cd7766ce7faddaa4 100644 (file)
@@ -58,11 +58,12 @@ DEF_TEST(usage_rate) {
 }
 
 DEF_TEST(usage_ratio) {
+  const size_t cpu_num = 4;
   usage_t usage = {0};
 
   cdtime_t t0 = TIME_T_TO_CDTIME_T(100);
   usage_init(&usage, t0);
-  for (size_t cpu = 0; cpu < 4; cpu++) {
+  for (size_t cpu = 0; cpu < cpu_num; cpu++) {
     for (state_t s = 0; s < STATE_ACTIVE; s++) {
       usage_record(&usage, cpu, s, 1000);
     }
@@ -73,7 +74,7 @@ DEF_TEST(usage_ratio) {
   derive_t global_increment = 0;
   derive_t state_increment[STATE_MAX] = {0};
 
-  for (size_t cpu = 0; cpu < 4; cpu++) {
+  for (size_t cpu = 0; cpu < cpu_num; cpu++) {
     for (state_t s = 0; s < STATE_ACTIVE; s++) {
       derive_t increment = ((derive_t)cpu * STATE_ACTIVE) + ((derive_t)s);
       usage_record(&usage, cpu, s, 1000 + increment);
@@ -89,7 +90,7 @@ DEF_TEST(usage_ratio) {
     }
   }
 
-  for (size_t cpu = 0; cpu < 4; cpu++) {
+  for (size_t cpu = 0; cpu < cpu_num; cpu++) {
     derive_t active_increment = 0;
     for (state_t s = 0; s < STATE_ACTIVE; s++) {
       derive_t increment = ((derive_t)cpu * STATE_ACTIVE) + ((derive_t)s);
@@ -111,7 +112,7 @@ DEF_TEST(usage_ratio) {
   }
 
   gauge_t sum = 0;
-  for (size_t cpu = 0; cpu < 4; cpu++) {
+  for (size_t cpu = 0; cpu < cpu_num; cpu++) {
     for (state_t s = 0; s < STATE_ACTIVE; s++) {
       gauge_t rate = usage_ratio(&usage, cpu, s);
       sum += rate;