keith/ForgePatch
1
0
Fork 0
Code Issues Pull requests Projects Releases 2 Wiki Activity

Add NPE protection to in-hand item rendering.

Browse source
This commit is contained in:
LexManos 2016-05-04 22:29:00 -07:00
parent 5f74a75435
commit 0d13fe245d
2 changed files with 24 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
patches/minecraft/net/minecraft/client/renderer/ItemRenderer.java.patch
View file

@ -1,5 +1,14 @@
--- ../src-base/minecraft/net/minecraft/client/renderer/ItemRenderer.java --- ../src-base/minecraft/net/minecraft/client/renderer/ItemRenderer.java
+++ ../src-work/minecraft/net/minecraft/client/renderer/ItemRenderer.java +++ ../src-work/minecraft/net/minecraft/client/renderer/ItemRenderer.java
@@ -316,7 +316,7 @@
{
ItemStack itemstack = abstractclientplayer.func_184607_cu();
- if (itemstack.func_77973_b() == Items.field_151031_f)
+ if (itemstack != null && itemstack.func_77973_b() == Items.field_151031_f) //Forge: Data watcher can desync and cause this to NPE...
{
EnumHand enumhand1 = abstractclientplayer.func_184600_cs();
flag = enumhand1 == EnumHand.MAIN_HAND;
@@ -360,7 +360,7 @@ @@ -360,7 +360,7 @@
this.func_187456_a(p_187457_7_, p_187457_5_, enumhandside); this.func_187456_a(p_187457_7_, p_187457_5_, enumhandside);
} }

35
patches/minecraft/net/minecraft/entity/EntityLivingBase.java.patch
View file

@ -232,33 +232,28 @@
public abstract EnumHandSide func_184591_cq(); public abstract EnumHandSide func_184591_cq();
public boolean func_184587_cr() public boolean func_184587_cr()
@@ -2369,8 +2435,16 @@ @@ -2371,12 +2437,19 @@
{
ItemStack itemstack = this.func_184586_b(this.func_184600_cs());
- if (itemstack == this.field_184627_bm) if (itemstack == this.field_184627_bm)
+ if (itemstack == this.field_184627_bm && itemstack != null)
{ {
+ field_184628_bn = net.minecraftforge.event.ForgeEventFactory.onItemUseTick(this, field_184627_bm, field_184628_bn); + if (this.field_184627_bm != null)
+ if (field_184628_bn <= 0)
+ { + {
+ this.func_71036_o(); + field_184628_bn = net.minecraftforge.event.ForgeEventFactory.onItemUseTick(this, field_184627_bm, field_184628_bn);
+ if (field_184628_bn > 0)
+ field_184627_bm.func_77973_b().onUsingTick(field_184627_bm, this, field_184628_bn);
+ } + }
+ else +
+ { // Forge Keep unindented to lower patch
+ field_184627_bm.func_77973_b().onUsingTick(field_184627_bm, this, field_184628_bn); //Forge Added
if (this.func_184605_cv() <= 25 && this.func_184605_cv() % 4 == 0) if (this.func_184605_cv() <= 25 && this.func_184605_cv() % 4 == 0)
{ {
this.func_184584_a(this.field_184627_bm, 5); this.func_184584_a(this.field_184627_bm, 5);
@@ -2380,6 +2454,7 @@ }
- if (--this.field_184628_bn == 0 && !this.field_70170_p.field_72995_K)
+ if (--this.field_184628_bn <= 0 && !this.field_70170_p.field_72995_K)
{ {
this.func_71036_o(); this.func_71036_o();
} }
+ } @@ -2394,8 +2467,10 @@
}
else
{
@@ -2394,8 +2469,10 @@
if (itemstack != null && !this.func_184587_cr()) if (itemstack != null && !this.func_184587_cr())
{ {
@ -270,7 +265,7 @@
if (!this.field_70170_p.field_72995_K) if (!this.field_70170_p.field_72995_K)
{ {
@@ -2478,6 +2555,8 @@ @@ -2478,6 +2553,8 @@
this.func_184584_a(this.field_184627_bm, 16); this.func_184584_a(this.field_184627_bm, 16);
ItemStack itemstack = this.field_184627_bm.func_77950_b(this.field_70170_p, this); ItemStack itemstack = this.field_184627_bm.func_77950_b(this.field_70170_p, this);
@ -279,7 +274,7 @@
if (itemstack != null && itemstack.field_77994_a == 0) if (itemstack != null && itemstack.field_77994_a == 0)
{ {
itemstack = null; itemstack = null;
@@ -2507,7 +2586,8 @@ @@ -2507,7 +2584,8 @@
{ {
if (this.field_184627_bm != null) if (this.field_184627_bm != null)
{ {
@ -289,7 +284,7 @@
} }
this.func_184602_cy(); this.func_184602_cy();
@@ -2626,4 +2706,28 @@ @@ -2626,4 +2704,28 @@
{ {
return true; return true;
} }