src: clarify the parameter name in Permission::Apply

This fixes confusing parameter names. They are references to set allow-permission. Signed-off-by: Daeyeon Jeong <daeyeon.dev@gmail.com>
nodejs · May 5, 2023 · 77416b8 · 77416b8
1 parent 3d00354
commit 77416b8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/permission/fs_permission.h b/src/permission/fs_permission.h
@@ -16,7 +16,7 @@ namespace permission {
 
 class FSPermission final : public PermissionBase {
  public:
-  void Apply(const std::string& deny, PermissionScope scope) override;
+  void Apply(const std::string& allow, PermissionScope scope) override;
   bool is_granted(PermissionScope perm, const std::string_view& param) override;
 
   // For debugging purposes, use the gist function to print the whole tree

diff --git a/src/permission/permission.h b/src/permission/permission.h
@@ -46,7 +46,7 @@ class Permission {
                                 const std::string_view& res);
 
   // CLI Call
-  void Apply(const std::string& deny, PermissionScope scope);
+  void Apply(const std::string& allow, PermissionScope scope);
   void EnablePermissions();
 
  private: