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

You derped a bit and forgot to remove Tile Entities #30

Merged
merged 1 commit into from
May 6, 2014

Conversation

wasliebob
Copy link
Contributor

You forgot to remove tiles on block break, if you don't want pr's feel free to ignore this.

You forgot to remove tiles on block break, if you don't want pr's feel free to ignore this.
@Aroma1997
Copy link
Owner

That may explain a lot...

Aroma1997 added a commit that referenced this pull request May 6, 2014
Removing Tile Entities...
@Aroma1997 Aroma1997 merged commit 04caad1 into Aroma1997:master May 6, 2014
@wasliebob
Copy link
Contributor Author

I might have derped myself a bit i think something else is wrong let me check it.

@sfPlayer1
Copy link

Actually that's not quite, right. The super call already handles the TE removal.

For IC2 integration I highly recommend switching to BasicSource, just like https://github.com/ForestryMC/ForestryMC/blob/master/forestry_common/energy/forestry/energy/gadgets/MachineGenerator.java

@Aroma1997
Copy link
Owner

Ok, so it doesn't explain a lot.
And I might look into that BasicSource, but ATM, I'm busy with RL stuff and have things higher on my todo list.

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.

3 participants