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
Cleanup #76
Cleanup #76
Conversation
src/main/java/micdoodle8/mods/galacticraft/api/transmission/tile/ITransmitter.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I definitely read all 25k changed lines. Joking aside, all the refactorings seem sensible to me.
@@ -256,8 +256,8 @@ public void onUpdate() { | |||
final List<?> var15 = this.worldObj.getEntitiesWithinAABBExcludingEntity(this, box); | |||
|
|||
if (var15 != null && !var15.isEmpty()) { | |||
for (int var52 = 0; var52 < var15.size(); ++var52) { | |||
final Entity var17 = (Entity) var15.get(var52); | |||
for (Object element : var15) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can now be Entity if you fix the generic type above
@@ -93,8 +93,8 @@ public void onUpdate() { | |||
.getEntitiesWithinAABBExcludingEntity(this, this.boundingBox.expand(1.0D, 0.5D, 1.0D)); | |||
|
|||
if (var3 != null) { | |||
for (int var4 = 0; var4 < var3.size(); ++var4) { | |||
final Entity var5 = (Entity) var3.get(var4); | |||
for (Object element : var3) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Entity
@@ -118,8 +118,8 @@ public void onUpdate() { | |||
.getEntitiesWithinAABBExcludingEntity(this, this.boundingBox.expand(1.0D, 0.5D, 1.0D)); | |||
|
|||
if (var3 != null) { | |||
for (int var4 = 0; var4 < var3.size(); ++var4) { | |||
final Entity var5 = (Entity) var3.get(var4); | |||
for (Object element : var3) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Entity
|
||
if (e instanceof EntityTieredRocket) { | ||
final EntityTieredRocket ship = (EntityTieredRocket) e; | ||
for (Object element : l) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Entity
|
||
if (e instanceof EntityTieredRocket) { | ||
final EntityTieredRocket ship = (EntityTieredRocket) e; | ||
for (Object element : l) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Entity
See @eigenraven's review of #76
* Fix oversights of last PR See @eigenraven's review of #76 * Update dependencies * Rework rocket fuel API
/gchelp
chat output via code and not by serializing JSONBlockCavernousVine#onEntityCollidedWithBlock
@API
annotations