Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the stickhelmet #25

Merged
merged 2 commits into from
Oct 4, 2024
Merged

Conversation

chochem
Copy link
Member

@chochem chochem commented Sep 28, 2024

sorry for the nerf.

these stickhelmets:
image
image

@chochem chochem requested a review from a team September 28, 2024 20:57
Copy link

@Alastors Alastors left a comment

Choose a reason for hiding this comment

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

How dare you 😭

@NotAPenguin0
Copy link

A tragic loss

@@ -26,7 +25,6 @@ public StickItem() {
@SideOnly(Side.CLIENT)
@Override
public IIcon getIconFromDamage(int meta) {
if (meta >= textureNames.length) return Items.leather_helmet.getIconFromDamage(meta);
Copy link
Member

Choose a reason for hiding this comment

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

You can use Util.getWithFallback a716d2b (#16)

Copy link
Member Author

Choose a reason for hiding this comment

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

I dont see why I would want to give textures to stuff that doesnt exist. most mods dont.

Copy link
Member Author

Choose a reason for hiding this comment

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

there always is the default fallback: the flame. and it clearly tells you that something is broken and needs fixing.

Copy link
Member

Choose a reason for hiding this comment

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

Well in the case I linked it crashed the game. As long as it doesn't, sure

@Dream-Master Dream-Master requested a review from miozune October 1, 2024 05:08
@chochem chochem merged commit a1ff81e into GTNewHorizons:master Oct 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants