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 ListManagerDummy::members(std::string list_address) + std::vector 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 lists(); - std::string moderator(std::string list_address); - std::vector members(std::string list_address); + std::string moderator(const std::string& list_address); + std::vector 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};