-
Notifications
You must be signed in to change notification settings - Fork 91
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
A variety of small bugfixes #609
Conversation
…he square root of a negative number
…by defaulting all readable items to read NullIota() if no iota contained. Fixed behaviour of OpRead and similar if no IotaHolder found.
…umptionPriority.
…from things with nothing in them), remove references to Hands in places where they aren't needed.
…adding a writeable() method to IotaHolderItem.
…ackets unnecessarily, getting names of patterns for every cast and only using if a mishap occurs)
… enough for that to be fine.
… longer triggering.
Merge conflict needs resolving, apparently |
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.
looks mostly fine to me. lots of file changes but they're all kinda changing a lot of the same things. Probably want to look at adding that translation key and not doing queryForMatchingStack
twice though
Common/src/main/java/at/petrak/hexcasting/api/casting/mishaps/MishapBadOffhandItem.kt
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/items/storage/ItemThoughtKnot.java
Outdated
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/items/storage/ItemFocus.java
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/casting/arithmetic/DoubleArithmetic.kt
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/casting/actions/spells/OpPlaceBlock.kt
Outdated
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.
ok, fixed the stuff i wanted fixed (it was all quite small stuff)
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.
Mostly looks good. Added a couple suggestions for clarification.
Common/src/main/java/at/petrak/hexcasting/api/casting/mishaps/MishapBadOffhandItem.kt
Outdated
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/casting/actions/rw/OpWrite.kt
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/casting/arithmetic/DoubleArithmetic.kt
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/items/storage/ItemFocus.java
Show resolved
Hide resolved
Common/src/main/java/at/petrak/hexcasting/common/items/storage/ItemFocus.java
Outdated
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.
looks gamer to me
Fixing several issues listed below, and a number of other issues that don't (I think) have issue trackers since I noticed them as I was fixing known issues.
Fix #608, fix #585, fix #551, fix #545, fix #541, fix #515, fix #513