From 7b83c29685736dc236a10279df50291de853fc19 Mon Sep 17 00:00:00 2001
From: Davin McCall <davmac@davmac.org>
Date: Sat, 20 Oct 2018 14:00:54 +0100
Subject: [PATCH] Fix minor (non-bug) issues picked up by cppcheck.

---
 src/includes/service.h | 5 ++---
 src/load-service.cc    | 2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/includes/service.h b/src/includes/service.h
index dd709ca..38c3098 100644
--- a/src/includes/service.h
+++ b/src/includes/service.h
@@ -421,8 +421,8 @@ class service_record
     public:
 
     service_record(service_set *set, string name)
-        : service_state(service_state_t::STOPPED), desired_state(service_state_t::STOPPED),
-            auto_restart(false), smooth_recovery(false),
+        : service_name(name), service_state(service_state_t::STOPPED),
+            desired_state(service_state_t::STOPPED), auto_restart(false), smooth_recovery(false),
             pinned_stopped(false), pinned_started(false), waiting_for_deps(false),
             waiting_for_console(false), have_console(false), waiting_for_execstat(false),
             start_explicit(false), prop_require(false), prop_release(false), prop_failure(false),
@@ -430,7 +430,6 @@ class service_record
             start_skipped(false), force_stop(false)
     {
         services = set;
-        service_name = name;
         record_type = service_type_t::DUMMY;
         socket_perms = 0;
     }
diff --git a/src/load-service.cc b/src/load-service.cc
index 37c00bc..1252c75 100644
--- a/src/load-service.cc
+++ b/src/load-service.cc
@@ -244,7 +244,7 @@ static void do_env_subst(std::string &line, std::list<std::pair<unsigned,unsigne
 // a fatal error.
 static void process_dep_dir(dirload_service_set &sset,
         const char *servicename,
-        string service_filename,
+        const string &service_filename,
         std::list<prelim_dep> &deplist, const std::string &depdirpath,
         dependency_type dep_type)
 {
-- 
2.25.1