Zero-length array allocation inspection (idea)

botany/src/main/java/binnie/botany/Botany.java

@@ -82,11 +82,6 @@ public IBinnieGUID[] getGUIDs() {
 		return BotanyGUI.values();
 	}
 
-	@Override
-	public Class<?>[] getConfigs() {
-		return new Class[0];
-	}
-
 	@Override
 	public IPacketID[] getPacketIDs() {
 		return PacketID.values();

botany/src/main/java/binnie/botany/genetics/FlowerBreedingSystem.java

@@ -41,7 +41,7 @@ public FlowerBreedingSystem() {
 	}
 
 	@Override
-	public float getChance(IMutation mutation, EntityPlayer player, IAlleleSpecies firstSpecies, IAlleleSpecies secondSpecies) {
+	public float getChance(final IMutation mutation, final EntityPlayer player, final IAlleleSpecies firstSpecies, final IAlleleSpecies secondSpecies) {
 		return mutation.getBaseChance();
 	}
 

botany/src/main/java/binnie/botany/genetics/FlowerRoot.java

@@ -209,11 +209,6 @@ public IAllele[] getDefaultTemplate() {
 		return alleles;
 	}
 
-	@Override
-	public IAllele[] getRandomTemplate(Random rand) {
-		return speciesTemplates.values().toArray(new IAllele[0][])[rand.nextInt(speciesTemplates.values().size())];
-	}
-
 	@Override
 	public ArrayList<IFlowerMutation> getMutations(boolean shuffle) {
 		if (shuffle) {

core/src/main/java/binnie/core/AbstractMod.java

@@ -19,6 +19,7 @@
 import binnie.core.network.IPacketProvider;
 import binnie.core.network.packet.MessageBinnie;
 import binnie.core.proxy.IProxyCore;
+import binnie.core.util.EmptyHelper;
 
 public abstract class AbstractMod implements IPacketProvider, IInitializable {
 	protected final List<IInitializable> modules;
@@ -40,15 +41,15 @@ public AbstractMod() {
 
 	@Override
 	public IPacketID[] getPacketIDs() {
-		return new IPacketID[0];
+		return EmptyHelper.PACKET_IDS_EMPTY;
 	}
 
 	public IBinnieGUID[] getGUIDs() {
-		return new IBinnieGUID[0];
+		return EmptyHelper.BINNIE_GUIDS_EMPTY;
 	}
 
 	public Class<?>[] getConfigs() {
-		return new Class[0];
+		return EmptyHelper.CLASS_ARRAY_EMPTY;
 	}
 
 	public abstract IProxyCore getProxy();

core/src/main/java/binnie/core/block/BlockMetadata.java

@@ -1,7 +1,6 @@
 package binnie.core.block;
 
 import javax.annotation.Nullable;
-import java.util.ArrayList;
 import java.util.List;
 
 import net.minecraft.block.Block;
@@ -24,42 +23,40 @@ public BlockMetadata(final Material material) {
 	}
 
 	public static ItemStack getBlockDropped(IBlockMetadata block, IBlockAccess world, BlockPos pos) {
-		TileEntityMetadata tile = TileEntityMetadata.getTile(world, pos);
+		final TileEntityMetadata tile = TileEntityMetadata.getTile(world, pos);
 		if (tile != null && !tile.hasDroppedBlock()) {
-			int meta = block.getDroppedMeta(world.getBlockState(pos), tile.getTileMetadata());
+			final int meta = block.getDroppedMeta(world.getBlockState(pos), tile.getTileMetadata());
 			return TileEntityMetadata.getItemStack((Block) block, meta);
 		}
 		return ItemStack.EMPTY;
 	}
 
 	public static void getDrops(NonNullList<ItemStack> drops, IBlockMetadata block, IBlockAccess world, BlockPos pos) {
-		ItemStack drop = getBlockDropped(block, world, pos);
+		final ItemStack drop = getBlockDropped(block, world, pos);
 		if (!drop.isEmpty()) {
 			drops.add(drop);
 		}
 	}
 
 	public static boolean breakBlock(IBlockMetadata blockMetadata, @Nullable EntityPlayer player, World world, BlockPos pos) {
-		List<ItemStack> drops = new ArrayList<>();
-		Block block = (Block) blockMetadata;
-		TileEntityMetadata tile = TileEntityMetadata.getTile(world, pos);
+		List<ItemStack> drops = null;
+		final Block block = (Block) blockMetadata;
+		final TileEntityMetadata tile = TileEntityMetadata.getTile(world, pos);
 		if (tile != null && !tile.hasDroppedBlock()) {
 			drops = block.getDrops(world, pos, world.getBlockState(pos), 0);
 		}
 		boolean hasBeenBroken = world.setBlockToAir(pos);
-		if (hasBeenBroken && !world.isRemote && drops.size() > 0 && (player == null || !player.capabilities.isCreativeMode)) {
+		if (hasBeenBroken && !world.isRemote && drops != null && (player == null || !player.capabilities.isCreativeMode)) {
 			for (ItemStack drop : drops) {
 				spawnAsEntity(world, pos, drop);
 			}
-			if (tile != null) {
-				tile.dropBlock();
-			}
+			tile.dropBlock();
 		}
 		return hasBeenBroken;
 	}
 
 	public static ItemStack getPickBlock(World world, BlockPos pos) {
-		Block block = world.getBlockState(pos).getBlock();
+		final Block block = world.getBlockState(pos).getBlock();
 		return getBlockDropped((IBlockMetadata) block, world, pos);
 	}
 

core/src/main/java/binnie/core/block/ItemMetadata.java

@@ -27,15 +27,15 @@ public boolean placeBlockAt(ItemStack stack, EntityPlayer player, World world, B
 		if (!(block instanceof IBlockMetadata)) {
 			return false;
 		}
-		int placedMeta = ((IBlockMetadata) block).getPlacedMeta(stack, world, pos, side);
+		final int placedMeta = ((IBlockMetadata) block).getPlacedMeta(stack, world, pos, side);
 		if (placedMeta < 0) {
 			return false;
 		}
 		if (!world.setBlockState(pos, newState, 3)) {
 			return false;
 		}
 		if (world.getBlockState(pos).getBlock() == block) {
-			TileEntityMetadata tile = TileEntityMetadata.getTile(world, pos);
+			final TileEntityMetadata tile = TileEntityMetadata.getTile(world, pos);
 			if (tile != null) {
 				tile.setTileMetadata(placedMeta, false);
 			}

core/src/main/java/binnie/core/block/TileEntityMetadata.java

@@ -40,8 +40,10 @@ public static ItemStack getItemStack(final Block block, final int damage) {
 		return itemStack;
 	}
 
+	private static final int MAX_ITEM_DAMAGE = 16387;
+
 	public static void setItemDamage(final ItemStack item, final int i) {
-		item.setItemDamage((i < 16387) ? i : 16387);
+		item.setItemDamage((i < MAX_ITEM_DAMAGE) ? i : MAX_ITEM_DAMAGE);
 		final NBTTagCompound tag = new NBTTagCompound();
 		tag.setInteger("meta", i);
 		item.setTagCompound(tag);
@@ -76,7 +78,7 @@ public NBTTagCompound getUpdateTag() {
 	@Override
 	public void readFromNBT(final NBTTagCompound nbt) {
 		super.readFromNBT(nbt);
-		if(world!=null)
+		if (world != null)
 			setTileMetadata(nbt.getInteger("meta"), world.isRemote);
 		else
 			this.meta = nbt.getInteger("meta");
@@ -97,7 +99,7 @@ public void setTileMetadata(final int meta, final boolean notify) {
 		if (this.meta != meta) {
 			this.meta = meta;
 			if (notify) {
-				IBlockState state = world.getBlockState(pos);
+				final IBlockState state = world.getBlockState(pos);
 				world.notifyBlockUpdate(pos, state, state, 3);
 			}
 		}

core/src/main/java/binnie/core/circuits/BinnieCircuit.java

@@ -14,8 +14,10 @@ public class BinnieCircuit implements ICircuit {
 	private final int limit;
 	private final List<String> tooltips;
 
+	private static final int TOOLTIPS_ARRAY_CAPACITY = 8;
+
 	public BinnieCircuit(final String uid, final int limit, final ICircuitLayout layout, final ItemStack itemStack) {
-		this.tooltips = new ArrayList<>();
+		this.tooltips = new ArrayList<>(TOOLTIPS_ARRAY_CAPACITY);
 		this.uid = "for.binnie.circuit." + uid;
 		this.limit = limit;
 		ChipsetManager.circuitRegistry.registerCircuit(this);

core/src/main/java/binnie/core/config/ConfigProperty.java

@@ -5,12 +5,14 @@
 import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;
 
+import org.apache.commons.lang3.StringUtils;
+
 @Retention(RetentionPolicy.RUNTIME)
 @Target({ElementType.FIELD})
 public @interface ConfigProperty {
 	String key();
 
-	String category() default "";
+	String category() default StringUtils.EMPTY;
 
 	String[] comment() default {};