Comments and TODOs

src/main/java/electroblob/wizardry/registry/WizardryAdvancementTriggers.java

@@ -26,6 +26,7 @@ public final class WizardryAdvancementTriggers {
 	public static final CustomAdvancementTrigger spell_failure = new CustomAdvancementTrigger("trigger_spell_failure");
 	public static final CustomAdvancementTrigger wand_levelup = new CustomAdvancementTrigger("trigger_wand_levelup");
 	public static final CustomAdvancementTrigger restore_imbuement_altar = new CustomAdvancementTrigger("restore_imbuement_altar");
+	// TODO: Make a proper imbuement altar trigger like the arcane workbench one
 	public static final CustomAdvancementTrigger create_elemental_armour = new CustomAdvancementTrigger("create_elemental_armour");
 
 	public static final StructureTrigger visit_structure = new StructureTrigger(new ResourceLocation(Wizardry.MODID, "visit_structure"));

src/main/java/electroblob/wizardry/tileentity/TileEntityStatue.java

@@ -46,7 +46,9 @@ public class TileEntityStatue extends TileEntity implements ITickable {
 	/**
 	 * The position within the petrified creature this particular tileentity holds. 1 is at the bottom.
 	 */
-	public int position = 1; // TODO: Remove this, there is no need for more than 1 TE per statue
+	// TODO: Remove this, there is no need for more than 1 TE per statue - actually, there is in the case of petrified
+	// 		 creatures, which should show the block breaking animation on all parts... except this is also broken!
+	public int position = 1;
 
 	public void setCreatureAndPart(EntityLiving entity, int position, int parts){
 		this.creature = entity;

src/main/java/electroblob/wizardry/util/ParticleBuilder.java

@@ -44,8 +44,9 @@ import java.util.Random;
 
 // Strictly speaking, this isn't a builder class in the traditional sense, because rather than returning the built
 // object at the end, it sends it to be processed instead and returns nothing. Additionally, unlike most builders
-// it's a singleton, because it's likely to be called very frequently and there's no point making a new instance
-// every time and clogging the heap with objects. It's also lazy, see the comment about builder variables below.
+// it's a singleton, because it's likely to be called very frequently and since this only happens from a single (client)
+// thread, there's no point making a new instance every time and clogging the heap with objects. It's also lazy, see
+// the comment about builder variables below.
 public final class ParticleBuilder {
 
 	/** The static instance of the particle builder. */