pid1: get rid of unit_supported() helper

Another case where "open code" is easier to read than the helper.

diff --git a/src/core/unit.c b/src/core/unit.c
index 700f38a6cd5a47e57166eec77edfd89cbf8b58a9..2cd58c47044f3a18812276abd80138768f472a91 100644 (file)
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -1791,7 +1791,7 @@ int unit_start(Unit *u) {
          * condition checks, so that we rather return condition check errors (which are usually not
          * considered a true failure) than "not supported" errors (which are considered a failure).
          */
-        if (!unit_supported(u))
+        if (!unit_type_supported(u->type))
                 return -EOPNOTSUPP;
 
         /* Let's make sure that the deps really are in order before we start this. Normally the job engine
@@ -1826,7 +1826,7 @@ bool unit_can_start(Unit *u) {
         if (u->load_state != UNIT_LOADED)
                 return false;
 
-        if (!unit_supported(u))
+        if (!unit_type_supported(u->type))
                 return false;
 
         /* Scope units may be started only once */
@@ -1875,7 +1875,7 @@ int unit_stop(Unit *u) {
 bool unit_can_stop(Unit *u) {
         assert(u);
 
-        if (!unit_supported(u))
+        if (!unit_type_supported(u->type))
                 return false;
 
         if (u->perpetual)

diff --git a/src/core/unit.h b/src/core/unit.h
index 7456f99efe3d03032d47504425a0f6540338794a..094a451f3ca5ebb710b3f2325084e9dab3c830fb 100644 (file)
--- a/src/core/unit.h
+++ b/src/core/unit.h
@@ -800,10 +800,6 @@ bool unit_is_unneeded(Unit *u);
 pid_t unit_control_pid(Unit *u);
 pid_t unit_main_pid(Unit *u);
 
-static inline bool unit_supported(Unit *u) {
-        return unit_type_supported(u->type);
-}
-
 void unit_warn_if_dir_nonempty(Unit *u, const char* where);
 int unit_fail_if_noncanonical(Unit *u, const char* where);