From: Lennart Poettering <lennart@poettering.net>
Date: Fri, 13 Nov 2015 17:46:12 +0000 (+0100)
Subject: core: simplify scope unit GC checking code a bit
X-Git-Tag: v228~16^2~4
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=700e2d63b7458e57203355f840bd1dcc1fb46645;p=platform%2Fupstream%2Fsystemd.git

core: simplify scope unit GC checking code a bit
---

diff --git a/src/core/scope.c b/src/core/scope.c
index 5f6527c..1953af1 100644
--- a/src/core/scope.c
+++ b/src/core/scope.c
@@ -402,15 +402,10 @@ static bool scope_check_gc(Unit *u) {
         /* Never clean up scopes that still have a process around,
          * even if the scope is formally dead. */
 
-        if (u->cgroup_path) {
-                int r;
+        if (!u->cgroup_path)
+                return false;
 
-                r = cg_is_empty_recursive(SYSTEMD_CGROUP_CONTROLLER, u->cgroup_path);
-                if (r <= 0)
-                        return true;
-        }
-
-        return false;
+        return cg_is_empty_recursive(SYSTEMD_CGROUP_CONTROLLER, u->cgroup_path) <= 0;
 }
 
 static void scope_notify_cgroup_empty_event(Unit *u) {
@@ -577,6 +572,7 @@ const UnitVTable scope_vtable = {
 
         .no_alias = true,
         .no_instances = true,
+        .can_transient = true,
 
         .init = scope_init,
         .load = scope_load,
@@ -611,7 +607,5 @@ const UnitVTable scope_vtable = {
         .bus_set_property = bus_scope_set_property,
         .bus_commit_properties = bus_scope_commit_properties,
 
-        .can_transient = true,
-
         .enumerate = scope_enumerate,
 };