From: Mariusz Szafranski <mariuszx.szafranski@intel.com>
Date: Mon, 5 Mar 2018 15:04:10 +0000 (+0000)
Subject: ipmi plugin: Remove duplicate/unused code and unneeded brackets.
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=8c9d64bf7615875b5030127b4df68a620e084f0d;p=collectd.git

ipmi plugin: Remove duplicate/unused code and unneeded brackets.

Change-Id: Ia4298f75f59e697df25d05091adbb556b64869e1
Signed-off-by: Mariusz Szafranski <mariuszx.szafranski@intel.com>
---

diff --git a/src/ipmi.c b/src/ipmi.c
index a8805866..59710c35 100644
--- a/src/ipmi.c
+++ b/src/ipmi.c
@@ -387,14 +387,6 @@ static int sensor_list_add(c_ipmi_instance_t *st, ipmi_sensor_t *sensor) {
         ipmi_sensor_get_sensor_direction(sensor),
         ipmi_sensor_get_event_support(sensor));
 
-  /* Both `ignorelist' and `sensor_name_ptr' may be NULL. */
-  if (ignorelist_match(st->ignorelist, sensor_name_ptr) != 0) {
-    INFO("ipmi plugin: sensor_list_add: Ignore sensor `%s` of `%s`, "
-         "because it is on ignore list.",
-         sensor_name_ptr, st->name);
-    return -1;
-  }
-
   /* FIXME: Use rate unit or base unit to scale the value */
 
   sensor_type = ipmi_sensor_get_sensor_type(sensor);
@@ -804,10 +796,9 @@ entity_sensor_update_handler(enum ipmi_update_e op,
         status = ipmi_sensor_add_discrete_event_handler(
             sensor, sensor_discrete_event_handler, st);
 
-      if (status) {
+      if (status)
         ERROR("Unable to add sensor %s event handler, status: %d", sensor_name,
               status);
-      }
     }
   } else if (op == IPMI_DELETED) {
     sensor_list_remove(st, sensor);