-
Notifications
You must be signed in to change notification settings - Fork 273
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
Item functions #587
base: dev
Are you sure you want to change the base?
Item functions #587
Conversation
I'll review this tonight or tomorrow. |
This looks to me a little too much like Aspect-Oriented Programming. How much use are stack traces going to be under this new regime? |
Do you think this could be better implemented in Java 9's Flow API? Also, I feel like it would make more sense to migrate to 1.13 first before doing such a massive change to our item/block type system. |
Closed because we're waiting for 1.13 and this branch will probably be rewritten by then |
Reopening this branch since it has become relevant again. |
This PR will remove the item and block hierarchy, which has many problems with conflicting item and block behaviors and creates a bad system for modding support, which we may want to pursue in the future.
It uses functions to provide default item and block functionality, and provides a simple system mods and plugins can use to extend this functionality and add their own items/blocks with unique functionality in the future.