Azimkin/Bukkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removed superfluous recalculation call; Fixes BUKKIT-3728

Browse Source 
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.
This commit is contained in:
Jeffrey Wardian 2013-03-11 12:31:44 -07:00 committed by EdGruberman
parent 923506ece3
commit c6f2ec2eee
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/bukkit/permissions/PermissionAttachment.java
View File

@ -90,7 +90,6 @@ public class PermissionAttachment {
*/ */
public void setPermission(Permission perm, boolean value) { public void setPermission(Permission perm, boolean value) {
setPermission(perm.getName(), value); setPermission(perm.getName(), value);
permissible.recalculatePermissions();
} }
/** /**
@ -114,7 +113,6 @@ public class PermissionAttachment {
*/ */
public void unsetPermission(Permission perm) { public void unsetPermission(Permission perm) {
unsetPermission(perm.getName()); unsetPermission(perm.getName());
permissible.recalculatePermissions();
} }
/** /**