fix: virtual destructor warning

casbin · Sep 5, 2023 · 7ebd848 · 7ebd848
1 parent c49bcfd
commit 7ebd848
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/.gitignore b/.gitignore
@@ -361,6 +361,7 @@ MigrationBackup/
 cmake-build/
 xcode-build/
 cmake-build*/
+build_support/
 
 # pip
 *.egg-info

diff --git a/casbin/persist/adapter.cpp b/casbin/persist/adapter.cpp
@@ -25,8 +25,8 @@
 namespace casbin {
 
 // LoadPolicyLine loads a text line as a policy rule to model.
-void LoadPolicyLine(std::string line, const std::shared_ptr<Model>& model) {
-    if (line == "" || line.find("#") == 0)
+void LoadPolicyLine(const std::string& line, const std::shared_ptr<Model>& model) {
+    if (line.empty() || line.find('#') == 0)
         return;
 
     std::vector<std::string> tokens = Split(line, ",", -1);

diff --git a/casbin/persist/adapter.h b/casbin/persist/adapter.h
@@ -25,7 +25,7 @@
 namespace casbin {
 
 // LoadPolicyLine loads a text line as a policy rule to model.
-void LoadPolicyLine(std::string line, const std::shared_ptr<Model>& model);
+void LoadPolicyLine(const std::string& line, const std::shared_ptr<Model>& model);
 
 /**
  * Adapter is the interface for Casbin adapters.
@@ -34,6 +34,8 @@ class Adapter {
 public:
     bool filtered;
 
+    virtual ~Adapter() = default;
+
     /**
      * LoadPolicy loads all policy rules from the storage.
      *

diff --git a/include/casbin/persist/adapter.h b/include/casbin/persist/adapter.h
@@ -25,7 +25,7 @@
 namespace casbin {
 
 // LoadPolicyLine loads a text line as a policy rule to model.
-void LoadPolicyLine(std::string line, const std::shared_ptr<Model>& model);
+void LoadPolicyLine(const std::string& line, const std::shared_ptr<Model>& model);
 
 /**
  * Adapter is the interface for Casbin adapters.
@@ -34,6 +34,8 @@ class Adapter {
 public:
     bool filtered;
 
+    virtual ~Adapter() = default;
+
     /**
      * LoadPolicy loads all policy rules from the storage.
      *