Jump to content

ASM retransform


NiceDev

Recommended Posts

Long story short:

You're doing it wrong.

 

Long version:

What are you trying to transform and why?

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

Long story short:

You're doing it wrong.

 

Long version:

What are you trying to transform and why?

 

Sorry, I'm an asm skid, because I don't understand it ._.

I override the method "shouldCombineTextures" in the "LayerArmorBase" class. Here is my source:

 

LayerArmorBaseTransformer: http://hastebin.com/oqavulijow.m

LayerArmorBaseClassVisitor: http://hastebin.com/ajeruricuh.axapta

ShouldCombineTexturesInstructionAdapter: http://hastebin.com/erusefepar.java

Link to comment
Share on other sites

Why are you transforming the class? Especially if you have no idea what you're doing...

I want, that the armor is red at damaging, too. In the new version (1.8) it has been changed.

This Transformer works, too, but I want to change it by changing the config in the config-gui I created.

Link to comment
Share on other sites

Please explain better. You want to make the armor you're wearing red?

There is no need for this you can just add/edit layers.

No, but the armor is not going to be red, when the entity gets damage. With this fix, it's going to be red.

 

Without fix:

SMpDWrm.png

 

With fix:

JzVEswi.png

Link to comment
Share on other sites

Yup, you dont need ASM for this, you can just replace the layer for rendering armor.

Always, ALWAYS use ASM as a last resort, you have adequate non-asm-hacky options to solve this.

Also, Is this an actual bug or is it a intended feature of vanilla minecraft?

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Link to comment
Share on other sites

Yup, you dont need ASM for this, you can just replace the layer for rendering armor.

Always, ALWAYS use ASM as a last resort, you have adequate non-asm-hacky options to solve this.

Also, Is this an actual bug or is it a intended feature of vanilla minecraft?

Just how?

Link to comment
Share on other sites

Yup, you dont need ASM for this, you can just replace the layer for rendering armor.

Always, ALWAYS use ASM as a last resort, you have adequate non-asm-hacky options to solve this.

Also, Is this an actual bug or is it a intended feature of vanilla minecraft?

Just how?

 

Like mentioned above, you would add / edit the armor layers / armor rendering. Try looking into the RenderPlayerEvent - all rendering for the player is done there, including armor rendering (pretty sure - look into ModelPlayer.class and RenderPlayer.class to find more info on the biped layers for the player). Also, look at LayerBipedArmor.class and LayerArmorBase.class. Maybe you could create a new player model with the player's skin, 3 / 4 pixel arms, and armor (except the armor is made red) and cancel the vanilla model in the RenderPlayerEvent. I've done similar, it can be done.

Development of Plugins [2012 - 2014] Development of Mods [2012 - Current]

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.