library: normalized 'extents_free_all' use across APIs

With the way those 'extents_free_all' guys were coded,
there's no real need to check for a NULL this->extents
before calling 'em. That's how <stat> already does it.

Signed-off-by: Jim Warner <james.warner@comcast.net>

diff --git a/proc/diskstats.c b/proc/diskstats.c
index a9d2c593ee7abc6a9df781636fa6063f8afd339f..c47dfad7d4de20c55f3c8ef3ea25aef3e9971837 100644 (file)
--- a/proc/diskstats.c
+++ b/proc/diskstats.c
@@ -691,8 +691,7 @@ static int diskstats_stacks_reconfig_maybe (
         memcpy(this->items, items, sizeof(enum diskstats_item) * numitems);
         this->items[numitems] = DISKSTATS_logical_end;
         this->numitems = numitems + 1;
-        if (this->extents)
-            diskstats_extents_free_all(this);
+        diskstats_extents_free_all(this);
         return 1;
     }
     return 0;

diff --git a/proc/slabinfo.c b/proc/slabinfo.c
index 73aa232e367d3490f34e0805eb72775f80b69716..4a6d7ed56b9edb0409b4aa92b5ee19b3393b4ee7 100644 (file)
--- a/proc/slabinfo.c
+++ b/proc/slabinfo.c
@@ -716,8 +716,7 @@ static int slabinfo_stacks_reconfig_maybe (
         memcpy(this->items, items, sizeof(enum slabinfo_item) * numitems);
         this->items[numitems] = SLABINFO_logical_end;
         this->numitems = numitems + 1;
-        if (this->extents)
-            slabinfo_extents_free_all(this);
+        slabinfo_extents_free_all(this);
         return 1;
     }
     return 0;