]> git.ipfire.org Git - thirdparty/collectd.git/commitdiff
resource_metrics: Fix an incorrect pointer type.
authorFlorian Forster <octo@collectd.org>
Sun, 24 Dec 2023 20:44:59 +0000 (21:44 +0100)
committerFlorian Forster <octo@collectd.org>
Thu, 28 Dec 2023 19:13:31 +0000 (20:13 +0100)
It's only ever used for a NULL comparison, so it doesn't really matter,
but it is unnecessarily confusing to use an incorrect type.

src/utils/resource_metrics/resource_metrics.c

index 8e49bcc9dd2cf352ae03fa72dd5c5c6fea7f42cb..1182445a65401f9aa299ecdd6c6851dd5fd70c32 100644 (file)
@@ -160,9 +160,8 @@ static int compare_metrics(metric_t const *a, metric_t const *b) {
 }
 
 static bool metric_exists(metric_family_t const *fam, metric_t const *m) {
-  metric_family_t *found =
-      bsearch(m, fam->metric.ptr, fam->metric.num, sizeof(*fam->metric.ptr),
-              (void *)compare_metrics);
+  metric_t *found = bsearch(m, fam->metric.ptr, fam->metric.num,
+                            sizeof(*fam->metric.ptr), (void *)compare_metrics);
   return found != NULL;
 }
 
@@ -170,7 +169,6 @@ static int insert_metrics(metric_family_t *fam, metric_list_t metrics) {
   int skipped = 0;
   for (size_t i = 0; i < metrics.num; i++) {
     metric_t const *m = metrics.ptr + i;
-
     if (metric_exists(fam, m)) {
 #if COLLECT_DEBUG
       strbuf_t buf = STRBUF_CREATE;