Rename/reverse set_auto_restart() to is_shutting_down().
authorDavin McCall <davmac@davmac.org>
Fri, 24 May 2019 11:49:41 +0000 (21:49 +1000)
committerDavin McCall <davmac@davmac.org>
Fri, 24 May 2019 11:49:41 +0000 (21:49 +1000)
src/includes/service.h
src/service.cc

index c532d97ac2e9113d8d7d6b4392015067f34a2914..9f55f32736e3ea1d6235e8466fd33f267fb326e9 100644 (file)
@@ -917,28 +917,16 @@ class service_set
         process_queues();
     }
     
-    void set_auto_restart(bool restart) noexcept
+    bool is_shutting_down() noexcept
     {
-        restart_enabled = restart;
+        return !restart_enabled;
     }
-    
-    bool get_auto_restart() noexcept
-    {
-        return restart_enabled;
-    }
-    
+
     shutdown_type_t get_shutdown_type() noexcept
     {
         return shutdown_type;
     }
 
-    // Set the shutdown type to the specified type, without issuing a shutdown order. If all services
-    // stop, the shutdown type determines the action that Dinit will take.
-    void set_shutdown_type(shutdown_type_t new_shutdown_type) noexcept
-    {
-        shutdown_type = new_shutdown_type;
-    }
-
     // Get an identifier for the run-time type of the service set (similar to typeid, but without
     // requiring RTTI to be enabled during compilation).
     virtual int get_set_type_id()
index 2fc07b3504a92fa7719ac59423966534a45c900e..40ab90b539be194c8b49efdae97127d9d5ee638a 100644 (file)
@@ -71,7 +71,7 @@ void service_record::stopped() noexcept
     }
 
     bool will_restart = (desired_state == service_state_t::STARTED)
-            && services->get_auto_restart();
+            && !services->is_shutting_down();
 
     for (auto & dependency : depends_on) {
         // we signal dependencies in case they are waiting for us to stop:
@@ -130,7 +130,7 @@ void service_record::stopped() noexcept
 bool service_record::do_auto_restart() noexcept
 {
     if (auto_restart) {
-        return services->get_auto_restart();
+        return !services->is_shutting_down();
     }
     return false;
 }