From c6f2ec2eee665e047fbaa284c4271aeeb1b6c624 Mon Sep 17 00:00:00 2001
From: Jeffrey Wardian <krinsdeath@hotmail.com>
Date: Mon, 11 Mar 2013 12:31:44 -0700
Subject: 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.
---
 src/main/java/org/bukkit/permissions/PermissionAttachment.java | 2 --
 1 file changed, 2 deletions(-)

(limited to 'src')

diff --git a/src/main/java/org/bukkit/permissions/PermissionAttachment.java b/src/main/java/org/bukkit/permissions/PermissionAttachment.java
index c42b6464..fe00258c 100644
--- a/src/main/java/org/bukkit/permissions/PermissionAttachment.java
+++ b/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();
     }
 
     /**
-- 
cgit v1.2.3