Open DaMatrix opened 2 years ago
I've tested this and it works well. You should mark this for review.
there is still some work required before i can merge this: most importantly, i need to make heightmap mode actually work with the abstracted API, which i'm currently doing on dev/abstract-heightmap-gen
.
once that's done, i also still want to do a last round of refactoring in order to minimize the amount of duplicated code between versions, and move more stuff to the :api
module to make it more useful.
This is a huge PR (which, in retrospect, probably should have been broken up into multiple smaller PRs). In combination with #117, it makes the vast majority of the codebase be able to operate entirely independently of the Minecraft version being used.
Additional major changes include:
T_SHIFT
and related constants are now toHeightmapConstants
/VoxelConstants
FEventBus
, which uses a familiar annotation-based event subscription system. It supports a few additional features not commonly found in other event buses, such as the ability to weakly reference a registered handler object, and generic parameter matching.@DebugOnly
annotation is now gone. Debug statistics and related features are now available outside of debug mode.Logger
, as are messages which appear in the client-side chat menu.With the exception of the
:gl
module and children (which are handled by #117), nearly everything is affected in some way.Known shortcomings: