From 94983c3ca75317a2c6c7e0c8ca88b4f2a2ad0f4c Mon Sep 17 00:00:00 2001
From: ElgarL <ElgarL@palmergames.com>
Date: Thu, 24 Nov 2011 13:26:30 +0000
Subject: Optimize sorting to speedup permission tests.

---
 .../src/org/anjocaido/groupmanager/data/DataUnit.java        | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

(limited to 'EssentialsGroupManager/src/org/anjocaido/groupmanager/data')

diff --git a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java
index 924da3616..25546a0ea 100644
--- a/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java
+++ b/EssentialsGroupManager/src/org/anjocaido/groupmanager/data/DataUnit.java
@@ -18,7 +18,7 @@ public abstract class DataUnit {
 
     private WorldDataHolder dataSource;
     private String name;
-    private boolean changed;
+    private boolean changed, sorted = false;
     private ArrayList<String> permissions = new ArrayList<String>();
 
     public DataUnit(WorldDataHolder dataSource, String name) {
@@ -91,6 +91,7 @@ public abstract class DataUnit {
 // for(StackTraceElement st: Thread.currentThread().getStackTrace()){
 // GroupManager.logger.finest(st.toString());
 // }
+        sorted = false;
         changed = true;
     }
 
@@ -135,8 +136,15 @@ public abstract class DataUnit {
     public ArrayList<String> getPermissionList() {
         return new ArrayList<String>(permissions);
     }
+    
+    public boolean isSorted() {
+    	return this.sorted;
+    }
 
     public void sortPermissions() {
-        Collections.sort(permissions, StringPermissionComparator.getInstance());
+        if (!isSorted()) {
+        	Collections.sort(permissions, StringPermissionComparator.getInstance());
+        	sorted = true;
+        }
     }
 }
\ No newline at end of file
-- 
cgit v1.2.3