From e741064461a1873e31e89feca6e5466515e101a8 Mon Sep 17 00:00:00 2001
From: heck <heck@pep.foundation>
Date: Mon, 19 Apr 2021 14:47:50 +0200
Subject: [PATCH] ListMangerDummy - some const for the methods

---
 src/listmanager_dummy.cc |  8 ++++----
 src/listmanager_dummy.hh | 13 +++++++------
 2 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/src/listmanager_dummy.cc b/src/listmanager_dummy.cc
index 56d50e5..1c47e96 100644
--- a/src/listmanager_dummy.cc
+++ b/src/listmanager_dummy.cc
@@ -76,14 +76,14 @@ namespace pEp {
     }
 
     // public
-    void ListManagerDummy::close_db()
+    void ListManagerDummy::close_db() const
     {
         pEpLogClass("called");
         db.close_db();
     }
 
     // public
-    void ListManagerDummy::delete_db()
+    void ListManagerDummy::delete_db() const
     {
         pEpLogClass("called");
         try {
@@ -188,7 +188,7 @@ namespace pEp {
     }
 
     // public
-    std::string ListManagerDummy::moderator(std::string list_address)
+    std::string ListManagerDummy::moderator(const std::string& list_address)
     {
         pEpLogClass("called");
         ensure_db_initialized();
@@ -216,7 +216,7 @@ namespace pEp {
     }
 
     // public
-    std::vector<std::string> ListManagerDummy::members(std::string list_address)
+    std::vector<std::string> ListManagerDummy::members(const std::string& list_address)
     {
         pEpLogClass("called");
         ensure_db_initialized();
diff --git a/src/listmanager_dummy.hh b/src/listmanager_dummy.hh
index ba6ed48..09661c6 100644
--- a/src/listmanager_dummy.hh
+++ b/src/listmanager_dummy.hh
@@ -12,19 +12,20 @@
 namespace pEp {
     class ListManagerDummy {
     public:
-        ListManagerDummy(const std::string& db_path);
+        ListManagerDummy() = delete;
+        explicit ListManagerDummy(const std::string& db_path);
         // Update
-        void list_add(const std::string& addr_list, const std::string& addr_mgr);
+        void list_add(const std::string& addr_list, const std::string& addr_mgr) ;
         void list_delete(const std::string& addr_list);
         void member_add(const std::string& addr_list, const std::string& addr_member);
         void member_remove(const std::string& addr_list, const std::string& addr_member);
         // Query
         std::vector<std::string> lists();
-        std::string moderator(std::string list_address);
-        std::vector<std::string> members(std::string list_address);
+        std::string moderator(const std::string& list_address);
+        std::vector<std::string> members(const std::string& list_address);
         // db
-        void close_db();
-        void delete_db();
+        void close_db() const;
+        void delete_db() const;
         // Logging
         static bool log_enabled;
         Adapter::pEpLog::pEpLogger logger{"ListManagerDummy", log_enabled};