formatting fun

src/net/citizensnpcs/npc/CitizensNPC.java

@@ -32,7 +32,7 @@ public class CitizensNPC implements NPC {
         this.name = name;
         this.character = character;
         manager = (CitizensNPCManager) CitizensAPI.getNPCManager();
-        id = manager.getUniqueID();
+        id = manager.generateUniqueId();
     }
 
     public CitizensNPC(int id, String name, Character character) {
@@ -59,11 +59,10 @@ public class CitizensNPC implements NPC {
 
     @Override
     public void addTrait(Trait trait) {
-        if (!hasTrait(trait.getClass())) {
+        if (!hasTrait(trait.getClass()))
             traits.put(trait.getClass(), trait);
-        } else {
+        else
             Messaging.debug("The NPC already has the trait '" + getTrait(trait.getClass()).getName() + "'.");
-        }
     }
 
     @Override
@@ -134,11 +133,10 @@ public class CitizensNPC implements NPC {
             return;
         }
 
-        if (mcEntity == null) {
+        if (mcEntity == null)
             mcEntity = manager.spawn(this, loc);
-        } else {
+        else
             manager.spawn(this, loc);
-        }
 
         // Set the location
         addTrait(new SpawnLocation(loc));
@@ -162,9 +160,8 @@ public class CitizensNPC implements NPC {
 
     @Override
     public void remove() {
-        if (isSpawned()) {
+        if (isSpawned())
             despawn();
-        }
         manager.remove(this);
     }
 

src/net/citizensnpcs/npc/CitizensNPCManager.java

@@ -77,7 +77,7 @@ public class CitizensNPCManager implements NPCManager {
         return spawned.contains(entity.getEntityId());
     }
 
-    public int getUniqueID() {
+    public int generateUniqueId() {
         int count = 0;
         while (true) {
             if (getNPC(count) == null)
@@ -89,7 +89,7 @@ public class CitizensNPCManager implements NPCManager {
 
     public CraftNPC spawn(NPC npc, Location loc) {
         if (spawned.contains(npc.getBukkitEntity().getEntityId()))
-            throw new IllegalStateException("already spawned");
+            throw new IllegalStateException("The NPC with ID '" + npc.getId() + "' is already spawned.");
         WorldServer ws = getWorldServer(loc.getWorld());
         CraftNPC mcEntity = new CraftNPC(getMinecraftServer(ws.getServer()), ws, npc.getFullName(),
                 new ItemInWorldManager(ws));
@@ -104,11 +104,10 @@ public class CitizensNPCManager implements NPCManager {
 
     public void despawn(NPC npc) {
         if (!spawned.contains(npc.getBukkitEntity().getEntityId()))
-            throw new IllegalStateException("already despawned");
+            throw new IllegalStateException("The NPC with ID '" + npc.getId() + "' is already despawned.");
         CraftNPC mcEntity = ((CitizensNPC) npc).getHandle();
-        for (Player player : Bukkit.getOnlinePlayers()) {
+        for (Player player : Bukkit.getOnlinePlayers())
             ((CraftPlayer) player).getHandle().netServerHandler.sendPacket(new Packet29DestroyEntity(mcEntity.id));
-        }
         Location loc = npc.getBukkitEntity().getLocation();
         getWorldServer(loc.getWorld()).removeEntity(mcEntity);
         npc.getTrait(SpawnLocation.class).setLocation(loc);

src/net/citizensnpcs/npc/trait/CitizensTraitManager.java

@@ -13,9 +13,8 @@ public class CitizensTraitManager implements TraitManager {
 
     @Override
     public Trait getTrait(String name) {
-        if (registered.get(name) == null) {
+        if (registered.get(name) == null)
             return null;
-        }
         return registered.get(name).create();
     }
 
@@ -27,7 +26,7 @@ public class CitizensTraitManager implements TraitManager {
     @Override
     public void registerTraitWithFactory(String name, Factory<? extends Trait> factory) {
         if (registered.get(name) != null)
-            throw new IllegalArgumentException("trait factory already registered");
+            throw new IllegalArgumentException("Trait factory already registered.");
         registered.put(name, factory);
     }
 

src/net/citizensnpcs/test/StorageTest.java

@@ -8,4 +8,4 @@ public class StorageTest {
     public void testYaml() {
 
     }
-}
+}

src/net/citizensnpcs/util/ByIdArray.java

@@ -49,17 +49,13 @@ public class ByIdArray<T> implements Iterable<T> {
         return (T) elementData[index];
     }
 
-    /*@SuppressWarnings("unchecked")
+    /*
-    public T[] toArray(T[] a) {
+     * @SuppressWarnings("unchecked") public T[] toArray(T[] a) { if (a.length <
-    	if (a.length < size)
+     * size) // Make a new array of a's runtime type, but my contents: return
-    		// Make a new array of a's runtime type, but my contents:
+     * (T[]) Arrays.copyOf(elementData, size, a.getClass());
-    		return (T[]) Arrays.copyOf(elementData, size, a.getClass());
+     * System.arraycopy(elementData, 0, a, 0, size); if (a.length > size)
-    	System.arraycopy(elementData, 0, a, 0, size);
+     * a[size] = null; return a; }
-    	if (a.length > size)
+     */
-    		a[size] = null;
-    	return a;
-    }
-    */
 
     @Override
     public Iterator<T> iterator() {
@@ -136,4 +132,4 @@ public class ByIdArray<T> implements Iterable<T> {
     public boolean contains(int index) {
         return elementData.length > index && elementData[index] != null;
     }
-}
+}