Added TODO for API classes needing investigation

src/api/java/appeng/api/client/BakingPipeline.java

@@ -32,7 +32,12 @@ import net.minecraft.block.state.IBlockState;
 import net.minecraft.client.renderer.block.model.IBakedModel;
 import net.minecraft.util.EnumFacing;
 
-
+/**
+ * TODO: Needs to be moved to an internal class. API is only allowed to contain interfaces and/or data.
+ *
+ * @deprecated
+ */
+@Deprecated
 public class BakingPipeline<F, T> implements BakingPipelineElement<F, T>
 {
 
@@ -43,6 +48,9 @@ public class BakingPipeline<F, T> implements BakingPipelineElement<F, T>
 		this.pipeline = ImmutableList.copyOf( pipeline );
 	}
 
+	/**
+	 * TODO: fix generics
+	 */
 	public List pipe( List things, IBakedModel parent, IBlockState state, EnumFacing side, long rand )
 	{
 		for( BakingPipelineElement pipe : pipeline )

src/api/java/appeng/api/movable/IMovableHandler.java

@@ -55,7 +55,7 @@ public interface IMovableHandler
 	 * 	@code
 	 * 	Chunk c = world.getChunkFromBlockCoords( x, z );
 	 * 	c.setChunkBlockTileEntity( x & 0xF, y + y, z & 0xF, tile );
-	 * 
+	 *
 	 * 	if( c.isChunkLoaded )
 	 * 	{
 	 * 		world.addTileEntity( tile );

src/api/java/appeng/api/networking/security/BaseActionSource.java

@@ -23,7 +23,9 @@
 
 package appeng.api.networking.security;
 
-
+/**
+ * TODO: Consider refactoring.
+ */
 public class BaseActionSource
 {
 

src/api/java/appeng/api/networking/security/MachineSource.java

@@ -23,7 +23,9 @@
 
 package appeng.api.networking.security;
 
-
+/**
+ * TODO: Consider refactoring.
+ */
 public class MachineSource extends BaseActionSource
 {
 

src/api/java/appeng/api/networking/security/PlayerSource.java

@@ -26,7 +26,9 @@ package appeng.api.networking.security;
 
 import net.minecraft.entity.player.EntityPlayer;
 
-
+/**
+ * TODO: Consider refactoring.
+ */
 public class PlayerSource extends BaseActionSource
 {
 

src/api/java/appeng/api/parts/LayerBase.java

@@ -35,6 +35,8 @@ import appeng.api.util.AEPartLocation;
 /**
  * All Layers must extends this, this get part implementation is provided to interface with the parts, however a real
  * implementation will be used at runtime.
+ *
+ * TODO: Consider removing and replacing with capabilities.
  */
 public abstract class LayerBase extends TileEntity // implements IPartHost
 {

src/api/java/appeng/api/parts/LayerFlags.java

@@ -23,7 +23,9 @@
 
 package appeng.api.parts;
 
-
+/**
+ * TODO: Consider removing and replacing with capabilities.
+ */
 public enum LayerFlags
 {
 

src/api/java/appeng/api/parts/SelectedPart.java

@@ -28,7 +28,8 @@ import appeng.api.util.AEPartLocation;
 
 
 /**
- * Reports a selected part from th IPartHost
+ * Reports a selected part from the IPartHost
+ *
  */
 public class SelectedPart
 {

src/api/java/appeng/api/storage/MEMonitorHandler.java

@@ -42,7 +42,11 @@ import appeng.api.storage.data.IItemList;
  * listeners.
  *
  * @param <StackType>
+ * @deprecated
+ *
+ * TODO: Needs to be redesigned to solve performance issues. Also should not be part of the API as class.
  */
+@Deprecated
 public class MEMonitorHandler<StackType extends IAEStack> implements IMEMonitor<StackType>
 {
 

src/api/java/appeng/api/util/AEAxisAlignedBB.java

@@ -24,6 +24,8 @@ import net.minecraft.util.math.AxisAlignedBB;
 
 /**
  * Mutable stand in for Axis Aligned BB, this was used to prevent GC Thrashing.. Related code could also be re-written.
+ *
+ * TODO: Replace with interface and maybe factory should it ever be needed for addons.
  */
 public class AEAxisAlignedBB
 {

src/api/java/appeng/api/util/AEPartLocation.java

@@ -94,7 +94,7 @@ public enum AEPartLocation
 
 	/**
 	 * 100% chance of success.
-	 * 
+	 *
 	 * @param side
 	 * @return proper Part Location for a facing enum.
 	 */

src/api/java/appeng/api/util/WorldCoord.java

@@ -31,6 +31,8 @@ import net.minecraft.util.math.BlockPos;
 /**
  * Represents a relative coordinate, either relative to another object, or
  * relative to the origin of a dimension.
+ *
+ * TODO: Consider replacing with {@link BlockPos}
  */
 public class WorldCoord
 {