diff options
author | Arne Juul <arnej@verizonmedia.com> | 2021-06-01 13:54:20 +0000 |
---|---|---|
committer | Arne Juul <arnej@verizonmedia.com> | 2021-06-02 06:40:23 +0000 |
commit | c3d3fc8e7aedba5fb5330692ae866bbb2906ecdc (patch) | |
tree | 8a34aee989e5cb5c3e0e5727d58242bdb189dfbe | |
parent | 939c9837cdec21fbe78d9740219138981ba025c2 (diff) |
avoid using "int" for boolean return value
-rw-r--r-- | configd/src/apps/sentinel/manager.cpp | 2 | ||||
-rw-r--r-- | configd/src/apps/sentinel/manager.h | 2 | ||||
-rw-r--r-- | configd/src/apps/sentinel/sentinel.cpp | 4 |
3 files changed, 4 insertions, 4 deletions
diff --git a/configd/src/apps/sentinel/manager.cpp b/configd/src/apps/sentinel/manager.cpp index 80dd2a3fda8..6e0ed78211c 100644 --- a/configd/src/apps/sentinel/manager.cpp +++ b/configd/src/apps/sentinel/manager.cpp @@ -109,7 +109,7 @@ Manager::doConfigure() } -int +bool Manager::doWork() { // Return true if there are any running services, false if not. diff --git a/configd/src/apps/sentinel/manager.h b/configd/src/apps/sentinel/manager.h index 9d3d14c6aeb..24bd67cbc49 100644 --- a/configd/src/apps/sentinel/manager.h +++ b/configd/src/apps/sentinel/manager.h @@ -55,7 +55,7 @@ public: Manager(Env &env); virtual ~Manager(); bool terminate(); - int doWork(); + bool doWork(); void updateActiveFdset(fd_set *fds, int *maxNum); }; diff --git a/configd/src/apps/sentinel/sentinel.cpp b/configd/src/apps/sentinel/sentinel.cpp index ed0e6c1cfe5..18d4dc28f8a 100644 --- a/configd/src/apps/sentinel/sentinel.cpp +++ b/configd/src/apps/sentinel/sentinel.cpp @@ -118,6 +118,6 @@ main(int argc, char **argv) } EV_STOPPING("config-sentinel", "normal exit"); - int rv = manager.terminate(); - return rv; + bool rv = manager.terminate(); + return rv ? EXIT_SUCCESS : EXIT_FAILURE; } |