Skip to content
This repository has been archived by the owner on May 26, 2018. It is now read-only.

Commit

Permalink
ModLoader is officially deprecated. It will all cease to be with 1.7.
Browse files Browse the repository at this point in the history
  • Loading branch information
cpw committed Oct 6, 2013
1 parent bc64cea commit 55525f6
Show file tree
Hide file tree
Showing 5 changed files with 126 additions and 7 deletions.
47 changes: 44 additions & 3 deletions client/net/minecraft/src/BaseMod.java
Expand Up @@ -42,10 +42,18 @@
import cpw.mods.fml.common.TickType;
import cpw.mods.fml.relauncher.SideOnly;


/**
* Bye bye ModLoader. Deprecated without direct replacement
* @author cpw
*
*/
@Deprecated
public abstract class BaseMod implements cpw.mods.fml.common.modloader.BaseModProxy
{
// CALLBACK MECHANISMS

@Deprecated
public final boolean doTickInGame(TickType tick, boolean tickEnd, Object... data)
{
Minecraft mc = FMLClientHandler.instance().getClient();
Expand All @@ -57,6 +65,7 @@ public final boolean doTickInGame(TickType tick, boolean tickEnd, Object... data
return true;
}

@Deprecated
public final boolean doTickInGUI(TickType tick, boolean tickEnd, Object... data)
{
Minecraft mc = FMLClientHandler.instance().getClient();
Expand Down Expand Up @@ -177,12 +186,14 @@ public final boolean handleCommand(String command, Object... data)
* @param id
* @param metadata
*/
@Deprecated
public int addFuel(int id, int metadata)
{
return 0;
}

@SideOnly(CLIENT)
@Deprecated
public void addRenderer(Map<Class<? extends Entity>, Render> renderers)
{
}
Expand All @@ -195,6 +206,7 @@ public void addRenderer(Map<Class<? extends Entity>, Render> renderers)
* @param chunkX
* @param chunkZ
*/
@Deprecated
public void generateNether(World world, Random random, int chunkX, int chunkZ)
{
}
Expand All @@ -207,6 +219,7 @@ public void generateNether(World world, Random random, int chunkX, int chunkZ)
* @param chunkX
* @param chunkZ
*/
@Deprecated
public void generateSurface(World world, Random random, int chunkX, int chunkZ)
{
}
Expand All @@ -219,6 +232,7 @@ public void generateSurface(World world, Random random, int chunkX, int chunkZ)
* @param y
* @param z
*/
@Deprecated
@SideOnly(CLIENT)
public GuiContainer getContainerGUI(EntityClientPlayerMP player, int containerID, int x, int y, int z)
{
Expand All @@ -228,6 +242,7 @@ public GuiContainer getContainerGUI(EntityClientPlayerMP player, int containerID
/**
* Return the name of your mod. Defaults to the class name
*/
@Deprecated
public String getName()
{
return getClass().getSimpleName();
Expand All @@ -236,6 +251,7 @@ public String getName()
/**
* Get your mod priorities
*/
@Deprecated
public String getPriorities()
{
return "";
Expand All @@ -244,9 +260,11 @@ public String getPriorities()
/**
* Return the version of your mod
*/
@Deprecated
public abstract String getVersion();

@SideOnly(CLIENT)
@Deprecated
public void keyboardEvent(KeyBinding event)
{

Expand All @@ -255,11 +273,13 @@ public void keyboardEvent(KeyBinding event)
/**
* Load your mod
*/
@Deprecated
public abstract void load();

/**
* Finish loading your mod
*/
@Deprecated
public void modsLoaded()
{
}
Expand All @@ -270,6 +290,7 @@ public void modsLoaded()
* @param player
* @param item
*/
@Deprecated
public void onItemPickup(EntityPlayer player, ItemStack item)
{
}
Expand All @@ -281,17 +302,20 @@ public void onItemPickup(EntityPlayer player, ItemStack item)
* @param minecraftInstance the client
* @return true to continue receiving ticks
*/
@Deprecated
@SideOnly(CLIENT)
public boolean onTickInGame(float time, Minecraft minecraftInstance)
{
return false;
}

@Deprecated
public boolean onTickInGame(MinecraftServer minecraftServer)
{
return false;
}

@Deprecated
@SideOnly(CLIENT)
public boolean onTickInGUI(float tick, Minecraft game, GuiScreen gui)
{
Expand All @@ -305,6 +329,7 @@ public boolean onTickInGUI(float tick, Minecraft game, GuiScreen gui)
* @param text
*/
@Override
@Deprecated
public void clientChat(String text)
{
}
Expand All @@ -314,6 +339,7 @@ public void clientChat(String text)
* @param handler
*/
@SideOnly(CLIENT)
@Deprecated
public void clientConnect(NetClientHandler handler)
{

Expand All @@ -324,35 +350,40 @@ public void clientConnect(NetClientHandler handler)
* @param handler
*/
@SideOnly(CLIENT)
@Deprecated
public void clientDisconnect(NetClientHandler handler)
{

}
/**
* Called client side to receive a custom payload for this mod
*
*
* NOTE: this method is not provided in Risugami's implementation of BaseMod!
*
* @param packet
*/
@Override
@Deprecated
public void receiveCustomPacket(Packet250CustomPayload packet)
{
}

@SideOnly(CLIENT)
@Deprecated
public void registerAnimation(Minecraft game)
{

}

@SideOnly(CLIENT)
@Deprecated
public void renderInvBlock(RenderBlocks renderer, Block block, int metadata, int modelID)
{

}

@SideOnly(CLIENT)
@Deprecated
public boolean renderWorldBlock(RenderBlocks renderer, IBlockAccess world, int x, int y, int z, Block block, int modelID)
{
return false;
Expand All @@ -362,11 +393,13 @@ public boolean renderWorldBlock(RenderBlocks renderer, IBlockAccess world, int x
* NOTE: this method is not provided in Risugami's implementation of BaseMod!
*/
@Override
@Deprecated
public void serverConnect(NetHandler handler) {

}

@Override
@Deprecated
public void serverCustomPayload(NetServerHandler handler, Packet250CustomPayload packet)
{

Expand All @@ -376,6 +409,7 @@ public void serverCustomPayload(NetServerHandler handler, Packet250CustomPayload
* NOTE: this method is not provided in Risugami's implementation of BaseMod!
*/
@Override
@Deprecated
public void serverDisconnect() {

}
Expand All @@ -386,6 +420,7 @@ public void serverDisconnect() {
* @param item
* @param matrix
*/
@Deprecated
public void takenFromCrafting(EntityPlayer player, ItemStack item, IInventory matrix)
{
}
Expand All @@ -396,6 +431,7 @@ public void takenFromCrafting(EntityPlayer player, ItemStack item, IInventory ma
* @param player
* @param item
*/
@Deprecated
public void takenFromFurnace(EntityPlayer player, ItemStack item)
{
}
Expand All @@ -413,27 +449,30 @@ public String toString()
* Called when a chat message is received. Return true to stop further processing
*/
@Override
@Deprecated
public void serverChat(NetServerHandler source, String message)
{
}
/**
* Called when a new client logs in.
*
*
* NOTE: this method is not provided in Risugami's implementation of BaseMod!
*
* @param player
*/
@Override
@Deprecated
public void onClientLogin(EntityPlayer player)
{
}

/**
* Called when a client logs out of the server.
*
*
* NOTE: this method is not provided in Risugami's implementation of BaseMod!
*/
@Override
@Deprecated
public void onClientLogout(INetworkManager mgr)
{

Expand All @@ -443,12 +482,14 @@ public void onClientLogout(INetworkManager mgr)
* Spawn the entity of the supplied type, if it is your mod's
*/
@SideOnly(CLIENT)
@Deprecated
public Entity spawnEntity(int entityId, World world, double scaledX, double scaledY, double scaledZ)
{
return null;
}

@SideOnly(CLIENT)
@Deprecated
public void clientCustomPayload(NetClientHandler handler, Packet250CustomPayload packet)
{

Expand Down
5 changes: 4 additions & 1 deletion client/net/minecraft/src/EntityRendererProxy.java
Expand Up @@ -5,7 +5,7 @@
* are made available under the terms of the GNU Lesser Public License v2.1
* which accompanies this distribution, and is available at
* http://www.gnu.org/licenses/old-licenses/gpl-2.0.html
*
*
* Contributors:
* cpw - implementation
*/
Expand All @@ -15,18 +15,21 @@
import net.minecraft.client.Minecraft;
import net.minecraft.client.renderer.EntityRenderer;

@Deprecated
public class EntityRendererProxy extends EntityRenderer
{
public static final String fmlMarker = "This is an FML marker";
private Minecraft game;

@Deprecated
public EntityRendererProxy(Minecraft minecraft)
{
super(minecraft);
game = minecraft;
}

@Override
@Deprecated
public void func_78480_b(float tick)
{
super.func_78480_b(tick);
Expand Down
7 changes: 6 additions & 1 deletion client/net/minecraft/src/MLProp.java
Expand Up @@ -20,30 +20,35 @@

/**
* @author cpw
*
*
*/
@Retention(value = RUNTIME)
@Target(value = FIELD)
@Deprecated
public @interface MLProp
{
/**
* Adds additional help to top of configuration file.
*/
@Deprecated
String info() default "";

/**
* Maximum value allowed if field is a number.
*/
@Deprecated
double max() default Double.MAX_VALUE;

/**
* Minimum value allowed if field is a number.
*/
@Deprecated
double min() default Double.MIN_VALUE;

/**
* Overrides the field name for property key.
*/
@Deprecated
String name() default "";

}

14 comments on commit 55525f6

@forresthopkinsa
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's about time!

Also, this is a historic moment. So... first!

@Pokefenn
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, quite interesting.
It really was pointless having modloader.
ALL HAIL FORGE!

@robin4002
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍
Modloader in the trash :D

@jeffreykog
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Into the trash it goes!

@231231231234234234
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, about time! 👍

@BURN447
Copy link

@BURN447 BURN447 commented on 55525f6 Oct 6, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@forresthopkinsa
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, really? ^.^

@xbony2
Copy link

@xbony2 xbony2 commented on 55525f6 Oct 6, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, there we go.
cvomm

@BURN447
Copy link

@BURN447 BURN447 commented on 55525f6 Oct 6, 2013 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cpw
Copy link
Contributor Author

@cpw cpw commented on 55525f6 Oct 6, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we had it because modloader mattered when FML started. FML started as a port of modloader to the server, as we saw that 1.3 was going to merge client and server and risu was not interested in doing it..

@Vazkii
Copy link
Contributor

@Vazkii Vazkii commented on 55525f6 Oct 6, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@forresthopkinsa - I doubt it. He isn't even using ModLoader.
Regardless. /cheer

@MFernflower
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

h313b794a

@LexManos
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's it, stop with the memes and bullshit.
This is not a place to spam stupid things. If you actually have a legitimate issue bring it up. Else yes we're all happy RML is dead. Lets leave it at that.

@JoneKone
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I quest it was easier than I thought.

Please sign in to comment.