From a2d72e265aaf0cceb0eb16d7f76730054e7ff439 Mon Sep 17 00:00:00 2001
From: Lennart Poettering <lennart@poettering.net>
Date: Tue, 15 Nov 2016 19:23:29 +0100
Subject: [PATCH] core: drop n_in_gc_queue field of Manager structure

We count the units in the GC queue with this, but actually never make use of
it, hence drop it.
---
 src/core/manager.c | 2 --
 src/core/manager.h | 1 -
 src/core/unit.c    | 6 +-----
 3 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/core/manager.c b/src/core/manager.c
index 52174eac07d..dc81af94922 100644
--- a/src/core/manager.c
+++ b/src/core/manager.c
@@ -1015,8 +1015,6 @@ static unsigned manager_dispatch_gc_queue(Manager *m) {
                 }
         }
 
-        m->n_in_gc_queue = 0;
-
         return n;
 }
 
diff --git a/src/core/manager.h b/src/core/manager.h
index 35172fdba98..aa3f95e8e08 100644
--- a/src/core/manager.h
+++ b/src/core/manager.h
@@ -229,7 +229,6 @@ struct Manager {
         int pin_cgroupfs_fd;
 
         int gc_marker;
-        unsigned n_in_gc_queue;
 
         /* Flags */
         ManagerExitCode exit_code:5;
diff --git a/src/core/unit.c b/src/core/unit.c
index da9bb58a521..df60a5bf04f 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -391,8 +391,6 @@ void unit_add_to_gc_queue(Unit *u) {
 
         LIST_PREPEND(gc_queue, u->manager->gc_queue, u);
         u->in_gc_queue = true;
-
-        u->manager->n_in_gc_queue++;
 }
 
 void unit_add_to_dbus_queue(Unit *u) {
@@ -570,10 +568,8 @@ void unit_free(Unit *u) {
         if (u->in_cleanup_queue)
                 LIST_REMOVE(cleanup_queue, u->manager->cleanup_queue, u);
 
-        if (u->in_gc_queue) {
+        if (u->in_gc_queue)
                 LIST_REMOVE(gc_queue, u->manager->gc_queue, u);
-                u->manager->n_in_gc_queue--;
-        }
 
         if (u->in_cgroup_queue)
                 LIST_REMOVE(cgroup_queue, u->manager->cgroup_queue, u);