Skip to content

Commit

Permalink
Nature Overhail, y u randomly depend on metadata for modded leaves. #…
Browse files Browse the repository at this point in the history
  • Loading branch information
bonii-xx committed Oct 6, 2014
1 parent 5933333 commit 92a1948
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/main/java/tconstruct/blocks/slime/SlimeLeaves.java
Expand Up @@ -68,7 +68,7 @@ public void registerBlockIcons (IIconRegister iconRegister)
@SideOnly(Side.CLIENT)
public IIcon getIcon (int side, int meta)
{
int tex = meta % 4;
int tex = meta % 1;
this.setGraphicsLevel(Minecraft.getMinecraft().gameSettings.fancyGraphics);
if (this.field_150121_P)
return fancyIcons[tex];
Expand Down

5 comments on commit 92a1948

@tmpillbox
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did you modulo by 1? This literally will return tex = meta

@bonii-xx
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, this wil literally return tex = 0. ;)

@UndeadZeratul
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not just set tex to 0 then? Why even use tex? Why not just return fancyIcons[0] or fastIcons[0]?

@bonii-xx
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lazyness. Why do the leaves even have that code to begin with? I assume more were planned.

@UndeadZeratul
Copy link
Contributor

Choose a reason for hiding this comment

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

I cannot comment on the why. I can also get behind the laziness excuse :)

Please sign in to comment.