fix: clippy cleaning

casbin · Nov 16, 2023 · 9ea6ece · 9ea6ece
1 parent 6cd09aa
commit 9ea6ece
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 10 deletions.
diff --git a/src/config.rs b/src/config.rs
@@ -151,8 +151,7 @@ impl Config {
         if section.is_empty() {
             section = DEFAULT_SECTION.to_owned();
         }
-        let section_value =
-            self.data.entry(section).or_insert_with(HashMap::new);
+        let section_value = self.data.entry(section).or_default();
 
         // if key not exists then insert, else update
         let key_value = section_value.get_mut(&option);

diff --git a/src/enforcer.rs b/src/enforcer.rs
@@ -100,7 +100,7 @@ impl EnforceContext {
 
 impl EventEmitter<Event> for Enforcer {
     fn on(&mut self, e: Event, f: fn(&mut Self, EventData)) {
-        self.events.entry(e).or_insert_with(Vec::new).push(f)
+        self.events.entry(e).or_default().push(f)
     }
 
     fn off(&mut self, e: Event) {
@@ -151,7 +151,7 @@ impl Enforcer {
             self.eft.new_stream(&e_ast.value, max(policy_len, 1));
         let m_ast_compiled = self
             .engine
-            .compile_expression(&escape_eval(&m_ast.value))
+            .compile_expression(escape_eval(&m_ast.value))
             .map_err(Into::<Box<EvalAltResult>>::into)?;
 
         if policy_len == 0 {
@@ -280,7 +280,7 @@ impl Enforcer {
             self.eft.new_stream(&e_ast.value, max(policy_len, 1));
         let m_ast_compiled = self
             .engine
-            .compile_expression(&escape_eval(&m_ast.value))
+            .compile_expression(escape_eval(&m_ast.value))
             .map_err(Into::<Box<EvalAltResult>>::into)?;
 
         if policy_len == 0 {

diff --git a/src/rbac/default_role_manager.rs b/src/rbac/default_role_manager.rs
@@ -53,15 +53,12 @@ impl DefaultRoleManager {
     ) -> NodeIndex<u32> {
         let domain = domain.unwrap_or(DEFAULT_DOMAIN);
 
-        let graph = self
-            .all_domains
-            .entry(domain.into())
-            .or_insert_with(StableDiGraph::new);
+        let graph = self.all_domains.entry(domain.into()).or_default();
 
         let role_entry = self
             .all_domains_indices
             .entry(domain.into())
-            .or_insert_with(HashMap::new)
+            .or_default()
             .entry(name.into());
 
         let vacant_entry = match role_entry {