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

Update ItemToolCore.java #66

Open
wants to merge 1 commit into
base: 1.12
Choose a base branch
from

Conversation

MauveCloud
Copy link

Small fix for block breaking speed.

I realize this is probably too minor to make it into the 1.12 builds, and isn't really worth jumping into Discord to mention, but I wanted you to be aware of it for the 1.14+ updates (since the issues tab is missing instead of read-only, I can't tell whether it was filed as a bug before). I mainly noticed this when trying to break rails with a flux-infused pickaxe, which was no faster than bare hands even when enchanted with Efficiency V (players who use Railcraft might not even notice that, since iirc that mod adds a crowbar to break rails with instead).

Small fix for block breaking speed.
@KingLemming
Copy link
Member

Huh. Interesting. Everything has changed in 1.14+ so this code actually doesn't even exist, but it's something I can look at w/ 1.12. Honestly some of the block breaking mechanics are so arcane that I'm not even sure which blocks that particular condition is meant to catch.

Thanks!

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.

2 participants