]> git.ipfire.org Git - thirdparty/collectd.git/commitdiff
Pass strcmp() to c_avl_create() as function pointer, not data one
authorEero Tamminen <eero.t.tamminen@intel.com>
Fri, 19 Jan 2024 16:09:09 +0000 (18:09 +0200)
committerEero Tamminen <eero.t.tamminen@intel.com>
Fri, 26 Jan 2024 15:52:38 +0000 (17:52 +0200)
Matches how strcmp() is passed to c_avl_create() elsewhere.

Reported by -Wpendantic.

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
src/redfish.c
src/utils/format_stackdriver/format_stackdriver.c
src/write_prometheus.c

index d2d77482071c6d3489d15cdaba56eb0df64e746e..c3cb85aa9a2d501760e62d896feeca24c714e13b 100644 (file)
@@ -275,7 +275,7 @@ static int redfish_preconfig(void) {
     goto error;
 
   /* Creating placeholder for queries */
-  ctx.queries = c_avl_create((void *)strcmp);
+  ctx.queries = c_avl_create((int (*)(const void *, const void *))strcmp);
   if (ctx.services == NULL)
     goto free_services;
 
index 0900361b3bc0dde1a713ffa6a77f6d5ee92ae230..6e2e4e226274ee8a6672746132eeaee5c875aae6 100644 (file)
@@ -492,13 +492,14 @@ sd_output_t *sd_output_create(sd_resource_t *res) /* {{{ */
     return NULL;
   }
 
-  out->staged = c_avl_create((void *)strcmp);
+  out->staged = c_avl_create((int (*)(const void *, const void *))strcmp);
   if (out->staged == NULL) {
     sd_output_destroy(out);
     return NULL;
   }
 
-  out->metric_descriptors = c_avl_create((void *)strcmp);
+  out->metric_descriptors =
+      c_avl_create((int (*)(const void *, const void *))strcmp);
   if (out->metric_descriptors == NULL) {
     sd_output_destroy(out);
     return NULL;
index fd663c4d0e6cfbd8db5b7f579d01a890fbad8ea8..430cf3b72668e8f48a55352426ded1f83bcb8263 100644 (file)
@@ -732,7 +732,7 @@ static int prom_config(oconfig_item_t *ci) {
 
 static int prom_init() {
   if (prom_metrics == NULL) {
-    prom_metrics = c_avl_create((void *)strcmp);
+    prom_metrics = c_avl_create((int (*)(const void *, const void *))strcmp);
     if (prom_metrics == NULL) {
       ERROR("write_prometheus plugin: c_avl_create() failed.");
       return -1;