commit 3afd63273a3959c0b1cd85c249e0fa2b114abd13
author Bart Van Assche <bvanassche@google.com> Mon Nov 21 03:18:19 2022 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Nov 21 03:18:19 2022 +0000
tree 26661e592b18cde64f99a307f91ce1fc2d92a270
parent 208f93042bba8994d6278a6672042fcd125e0410
parent bfcf4374ed78e20394edb76b54e20bea4887b5cf

Merge "init: Document which class Service members are not modified"

init/service.h

diff --git a/init/service.h b/init/service.h
index b2c9909..e7211ec 100644
--- a/init/service.h
+++ b/init/service.h
@@ -143,7 +143,7 @@
             flags_ &= ~SVC_ONESHOT;
         }
     }
-    Subcontext* subcontext() const { return subcontext_; }
+    const Subcontext* subcontext() const { return subcontext_; }
     const std::string& filename() const { return filename_; }
     void set_filename(const std::string& name) { filename_ = name; }
 
@@ -162,7 +162,7 @@
     static std::chrono::time_point<std::chrono::steady_clock> exec_service_started_;
     static pid_t exec_service_pid_;
 
-    std::string name_;
+    const std::string name_;
     std::set<std::string> classnames_;
 
     unsigned flags_;
@@ -186,7 +186,7 @@
     // Environment variables that only get applied to the next run.
     std::vector<std::pair<std::string, std::string>> once_environment_vars_;
 
-    Subcontext* subcontext_;
+    const Subcontext* const subcontext_;
     Action onrestart_;  // Commands to execute on restart.
 
     std::vector<std::string> writepid_files_;
@@ -220,7 +220,7 @@
 
     bool updatable_ = false;
 
-    std::vector<std::string> args_;
+    const std::vector<std::string> args_;
 
     std::vector<std::function<void(const siginfo_t& siginfo)>> reap_callbacks_;