perf: Performance Enhancement (#144)

* perf: Clear the list of functions for every new duktape context Signed-off-by: Laszló Szabó <[email protected]> * perf: Split should only reserve vector of limit is set Signed-off-by: Laszló Szabó <[email protected]> Co-authored-by: Laszló Szabó <[email protected]>
casbin · Aug 23, 2021 · 8891fa9 · 8891fa9
1 parent 4dd852d
commit 8891fa9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/casbin/enforcer.cpp b/casbin/enforcer.cpp
@@ -38,6 +38,7 @@ namespace casbin {
 // use model matcher by default when matcher is "".
 bool Enforcer::m_enforce(const std::string& matcher, Scope scope) {
     m_func_map.scope = scope;
+    m_func_map.func_list.clear();
 
     if(!m_enabled)
         return true;

diff --git a/casbin/util/split.cpp b/casbin/util/split.cpp
@@ -33,9 +33,7 @@ std::vector<std::string> Split(std::string str, const std::string& del, int limi
 
     if(limit <= 0)
         limit = LARGE;
-
-    tokens.reserve((limit == LARGE) ? 100000 : limit);
-
+    tokens.reserve((limit == LARGE) ? 0 : limit);
     for (int i = 1; i < limit ; i++) {
         size_t pos = str.find(del);
         if (pos != std::string::npos) {