Draco18s

And sync the data to the client, of course.

There also shouldn't be empty stacks in the drops array.

No, what you need to do is extract the 2 bits of metadata. EnumFacing.getHorizontal(meta&3)

*cough, did you read this, cough*

return this.getDefaultState().withProperty(FACING, EnumFacing.getHorizontal(meta)) This line won't work if meta > 4 (which BARREL and SPIGOT make it be)

Basically, you'll collide from some directions and not others and general wonkiness. The only, the only, direction it is safe to extend a collision box outside the 1x1x1 cube is +Y and even then only an extra half-block.

If your block has a collision bounds greater than 1x1x1 you're in trouble there too: it won't work either. Getting fences to work properly was a problem Mojang took several versions to get right.

It is a little messy, and facings can be mitigated. You can convert a facing (EnumWest) into x and z offsets. "Forward" gets translated to facing.offsetX() + facing.offsetZ() because a facing will only ever have one of those values as non-zero. For right, you use facing.rotate(UP).offsetX() + facing.rotate(UP).offsetZ() (you can store the rotated facings as their own variables as well).

The functionailiy will continue to exist as Minecraft evolves, but it is possible that the method does get removed and placed somewhere else. It will depend on how Mojang continues to work with the IBlockState format.

Basically not really. The method is marked deprecated which means do not call this method. You should still override it to get the desired behavior.

Well, with a well-defined structure I wouldn't use getAllInBox. I'd go "this is the core, so at x+1 I should find BlockY, at x+2 I should find... And do a big query that way.

FastTESR is still slower than IBakedModel, but it's more performant than a regular TESR

ModelLoader.setCustomModelResourceLocation(item, metadata, modelResourceLocation); Hmm...

For this you should use a FastTESR. There is an animation API in JSON, but it's limited.

Why do you need a TESR (fast or otherwise) to render a block? Use the JSON model system.

Maybe you should ask the mod developer. We here don't even know what mod you're using because you conveniently left that out, must less know anything about it.

public/private change wasn't necessary, it was that the local-scope declaration overrode the parent scope declaration.

repair.isItemEqual(new ItemStack(RepairItem)) can be simplified torepair.getItem() == RepairItem Except that RepairItem is always null because you did this: Item RepairItem = repairitem; That variable is local and discarded as soon as the constructor finishes and theprivate Item RepairItem;declared is left undefined.

1.7.10 is no longer supported here.

There are a lot of things I don't know. I didn't start programming on Java so I generally take what I do know and make it work, it isn't until later that someone says, "hey, there's an easier way."

Cool. Could have used that four years ago, oh well.

This is old and wrong. 20 armor will stop 8 of 20 damage (40%). The more damage you're taking in a single hit, the less effective your armor is.

You mean as: effect.hearts=\0x2665 ? Orr...?

The point was to allow the display to be customizable because lang files are resource-pack-capable.

http://minecraft.gamepedia.com/Armor#Armor_Toughness