From 69740791f7d7aa7b84dcd2db0a45f5940a414448 Mon Sep 17 00:00:00 2001 From: Bukkit/Spigot Date: Mon, 11 Mar 2013 12:31:44 -0700 Subject: [PATCH] Removed superfluous recalculation call; Fixes BUKKIT-3728 The permission attachment interface provides two methods each for setting and unsetting permissions. Each one also provides an extra call to the recalculatePermissions() method on the permissible, which degrades performance. This commit removes the duplicate call to recalculate permissions. By: Jeffrey Wardian --- .../main/java/org/bukkit/permissions/PermissionAttachment.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/paper-api/src/main/java/org/bukkit/permissions/PermissionAttachment.java b/paper-api/src/main/java/org/bukkit/permissions/PermissionAttachment.java index c42b64642..fe00258c3 100644 --- a/paper-api/src/main/java/org/bukkit/permissions/PermissionAttachment.java +++ b/paper-api/src/main/java/org/bukkit/permissions/PermissionAttachment.java @@ -90,7 +90,6 @@ public class PermissionAttachment { */ public void setPermission(Permission perm, boolean value) { setPermission(perm.getName(), value); - permissible.recalculatePermissions(); } /** @@ -114,7 +113,6 @@ public class PermissionAttachment { */ public void unsetPermission(Permission perm) { unsetPermission(perm.getName()); - permissible.recalculatePermissions(); } /**