Ok, so I was looking through my key bindings code when I found something a bit questionable.

 	@SubscribeEvent
    public void onKeyInput(InputEvent.KeyInputEvent event) {
        if(KeyBindings.start.isPressed()){
        	PlayerUtils.startPlayer(Minecraft.getMinecraft().thePlayer);
        }
    }

 

The event itself isn't the problem, Minecraft.getMinecraft().thePlayer is the questionable part. That is client side code right? So I guess what I'm getting at is the question above: Are events client side or server side?

Most events are referenced on both sides, but since the server can't pick up on a specific player's key presses, it should never technically get called on the server. It would probably be a good idea to insert a check just to be safe though.

Most events are referenced on both sides, but since the server can't pick up on a specific player's key presses, it should never technically get called on the server. It would probably be a good idea to insert a check just to be safe though.

 

Ok so how do I insert a check for that?

if(FMLCommonHandler.instance().getSide().equals(Side.CLIENT)){
    //do client side stuff here
}

 

Be wary though, although I have never had problems with it, I have heard of people having issues where it returns true on an integrated server.

Ok, that will do. I guess if it has those problems it will just have an error.

 

Edit: Oh wait, on the event could I add a @SideOnly(Side.CLIENT) annotation? Would that work?

Ok, that will do. I guess if it has those problems it will just have an error.

 

Edit: Oh wait, on the event could I add a @SideOnly(Side.CLIENT) annotation? Would that work?

 

No.

http://www.minecraftforge.net/forum/index.php/topic,22764.0.html

Ok that makes sense. Thanks for the help everyone.