From: Florian Forster <octo@collectd.org>
Date: Fri, 19 Jan 2024 19:58:25 +0000 (+0100)
Subject: cpu plugin: Use a const instead of repeating a literal multiple times.
X-Git-Tag: 6.0.0-rc0~5^2~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2913513a10f1284b76fc4448bf39c21154ba6c55;p=thirdparty%2Fcollectd.git

cpu plugin: Use a const instead of repeating a literal multiple times.
---

diff --git a/src/cpu_test.c b/src/cpu_test.c
index ec4ed1b4f..ac5c4bf46 100644
--- a/src/cpu_test.c
+++ b/src/cpu_test.c
@@ -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;