FallFlyingLib

I am working on BetterEnd. We tried to fix compat issues with the Origin Mod, and just noticed that FallFlyingLib is using a @Redirect to apply custom changes to tickFallFlying.

FallFlyingLib/src/main/java/net/adriantodt/fallflyinglib/mixin/LivingEntityMixin.java

Lines 25 to 31 in b281b48

    
           @Redirect( 
        
               at = @At( 
        
                   value = "INVOKE", 
        
                   target = "Lnet/minecraft/entity/LivingEntity;tickFallFlying()V" 
        
               ), method = "tickMovement" 
        
           ) 
        
           public void fallflyinglib_flightPipeline(LivingEntity thisObj) {

This makes the library incompatible with any other Mod that modifies the behaviour of tickFallFlying. I had a look at the Code, and it should be easily replaceable with a canceling HEAD-inject to tickFallFlying. Would you please consider to change the Mixin? That would make the Lib much more compatible.

Honestly, I'd prefer to kill FFL from existance.

Mistakes were made regarding my choices with FFL, and I started creating a better version of FFL, this time with the ultimate goal to PR all the code to Fabric API. Sadly, I don't have enough time to finish it.

https://github.com/CafeteriaGuild/elytra-pipeline

I'd much more prefer to have a better, Fabric API-native way to handle all of this, and to axe down FFL from existance.

FallFlyingLib will (hopefully) be superseded by an official Fabric API module: FabricMC/fabric#1815

FallFlyingLib was superseded by the official Elytra events available at Fabric Entity API.

	@Redirect(
	at = @At(
	value = "INVOKE",
	target = "Lnet/minecraft/entity/LivingEntity;tickFallFlying()V"
	), method = "tickMovement"
	)
	public void fallflyinglib_flightPipeline(LivingEntity thisObj) {

Share to