jarvis... slime out flxanimate and replace him with flixel-animate. On it sir!

Author: CrusherNotDrip

.hxpkg

@@ -20,9 +20,9 @@
 				"branch": "technotdrip-engine"
 			},
 			{
-				"name": "flxanimate",
-				"link": "https://github.com/Dot-Stuff/flxanimate",
-				"branch": "ef85c132044d897a04c5008da29674d88f83c4d6"
+				"name": "flixel-animate",
+				"version": "1.3.1"
+			},
 			},
 			{
 				"name": "hxcpp",

Project.hxp

@@ -135,7 +135,7 @@ class Project extends HXProject
 		addHaxelib('openfl', '9.4.1');
 		addHaxelib('flixel', '6.1.0');
 		addHaxelib('flixel-addons', 'git');
-		addHaxelib('flxanimate', 'git');
+		addHaxelib('flixel-animate', '1.3.1');
 		addHaxelib('flixel-controls', 'git');
 		addHaxelib('extension-androidtools', '2.1.1', platformType == MOBILE && haxedefs.get('android'));
 		addHaxelib('hxcpp-debug-server', '1.2.4', debug);

src/funkin/objects/FunkinSprite.hx

@@ -1,20 +1,20 @@
 package funkin.objects;
 
+import animate.FlxAnimate;
+import animate.FlxAnimateFrames;
 import flixel.FlxCamera;
 import flixel.group.FlxSpriteGroup.FlxTypedSpriteGroup;
 import flixel.math.FlxPoint;
 import flixel.util.FlxSignal.FlxTypedSignal;
-import flxanimate.FlxAnimate;
 import funkin.structures.ObjectStructure;
 
 /**
  * This is a sprite class that adds on to the already existing FlxSprite.
  */
-class FunkinSprite extends FlxSprite
+class FunkinSprite extends FlxAnimate
 {
 	/**
 	 * uh oh! the letters are ANGRY!!
-	 * wont work on animate atlases :(
 	 */
 	static var spritesAreAngy:Bool = FlxG.random.bool(0.9125);
 
@@ -24,16 +24,16 @@ class FunkinSprite extends FlxSprite
 	 */
 	public var doInvisibleDraw:Bool = false;
 
-	/**
-	 * The Animate Atlas Object if the animation is one.
-	 */
-	public var atlas:FlxAnimate;
-
 	public function new(x:Float = 0, y:Float = 0)
 	{
 		super(x, y);
 	}
 
+	override function initVars():Void
+	{
+		super.initVars();
+	}
+
 	/**
 	 * Loads or creates a texture and applies it to this sprite.
 	 * @param path The asset path. (If `path` starts with a **#** then a color will be made instead and rectWidth + rectHeight will determine its size.)
@@ -63,58 +63,29 @@ class FunkinSprite extends FlxSprite
 	 */
 	public function loadFrames(path:String, ?forcedType:Null<String>):FunkinSprite
 	{
-		if (atlas != null)
-		{
-			atlas.destroy();
-			atlas = null;
-		}
-
-		// TODO: make forcedType work, im too lazy
-
 		if (Paths.location.exists(path + '.xml'))
 		{
 			frames = Paths.content.sparrowAtlas(path);
 		}
 		else if (Paths.location.exists(path + '/Animation.json'))
 		{
-			atlas = new FlxAnimate(0, 0, Paths.location.get(path), {
-				ShowPivot: false
-			});
+			frames = FlxAnimateFrames.fromAnimate(Paths.location.get(path));
 		}
 
 		return this;
 	}
 
-	override public function destroy():Void
-	{
-		if (atlas != null)
-			atlas.destroy();
-
-		super.destroy();
-	}
-
 	override public function draw():Void
 	{
 		var oldAlpha:Float = alpha;
 		if (doInvisibleDraw)
 			alpha = 0.0001;
 
-		if (atlas != null)
-		{
-			updateAtlasDummy();
-			atlas.draw();
-		}
-		else
-		{
-			super.draw();
-		}
+		super.draw();
 
 		if (doInvisibleDraw)
 		{
 			alpha = oldAlpha;
-
-			if (atlas != null)
-				atlas.alpha = alpha;
 		}
 	}
 
@@ -128,38 +99,6 @@ class FunkinSprite extends FlxSprite
 	}
 	#end
 
-	override public function update(elapsed:Float):Void
-	{
-		if (atlas != null)
-			atlas.update(elapsed);
-
-		super.update(elapsed);
-	}
-
-	/**
-	 * Updates the Atlas dummy's values, so it looks like it belongs to this sprite.
-	 * @see The Original Code: https://github.com/CodenameCrew/CodenameEngine/blob/f6deda2c84984202effdfc5f6b577c2d956aa7b5/source/funkin/backend/FunkinSprite.hx#L209C2-L232C3
-	 */
-	@:privateAccess
-	public function updateAtlasDummy():Void
-	{
-		atlas.cameras = cameras;
-		atlas.scrollFactor = scrollFactor;
-		atlas.scale = scale;
-		atlas.offset = offset;
-		atlas.x = x;
-		atlas.y = y;
-		atlas.angle = angle;
-		atlas.alpha = alpha;
-		atlas.visible = visible;
-		atlas.flipX = flipX;
-		atlas.flipY = flipY;
-		atlas.shader = shader;
-		atlas.antialiasing = antialiasing;
-		atlas.colorTransform = colorTransform;
-	}
-
-	// crusher, do you FUCKING dare import these.
 	override public function getScreenPosition(?result:FlxPoint, ?camera:FlxCamera):FlxPoint
 	{
 		var point:FlxPoint = super.getScreenPosition(result, camera);
@@ -191,11 +130,7 @@ class FunkinSprite extends FlxSprite
 		if (animationStunned)
 			return;
 
-		if (atlas != null)
-			atlas.anim.play(name, restart, reversed);
-		else
-			animation.play(name, restart, reversed);
-
+		animation.play(name, restart, reversed);
 		animationStunned = stunAnimations;
 		currentAnim = name;
 	}
@@ -210,19 +145,27 @@ class FunkinSprite extends FlxSprite
 	 */
 	public function addAnimation(name:String, anim:String, ?indices:Array<Int> = null, ?frameRate:Float = 24, ?looped:Bool = true):Void
 	{
-		if (atlas != null)
+		if (indices != null && indices.length > 0)
 		{
-			if (indices != null && indices.length > 0)
-				atlas.anim.addBySymbolIndices(name, anim + '\\', indices, frameRate, looped);
+			if (isAnimate)
+			{
+				this.anim.addBySymbolIndices(name, anim, indices, frameRate, looped);
+			}
 			else
-				atlas.anim.addBySymbol(name, anim + '\\', frameRate, looped);
+			{
+				animation.addByIndices(name, anim + '0', indices, '', frameRate, looped);
+			}
 		}
 		else
 		{
-			if (indices != null && indices.length > 0)
-				animation.addByIndices(name, anim + '0', indices, '', frameRate, looped);
+			if (isAnimate)
+			{
+				this.anim.addBySymbol(name, anim, frameRate, looped);
+			}
 			else
+			{
 				animation.addByPrefix(name, anim + '0', frameRate, looped);
+			}
 		}
 	}
 
@@ -233,7 +176,7 @@ class FunkinSprite extends FlxSprite
 
 	function get_animationIsNull():Bool
 	{
-		return (atlas != null) ? atlas.anim.curSymbol == null : animation.curAnim == null;
+		return animation.curAnim == null;
 	}
 
 	/**
@@ -243,10 +186,7 @@ class FunkinSprite extends FlxSprite
 
 	function get_animFinished():Bool
 	{
-		if (animationIsNull)
-			return false;
-
-		return ((atlas != null) ? atlas.anim.finished : animation.curAnim.finished) ?? false;
+		return animation?.curAnim?.finished ?? false;
 	}
 
 	/**
@@ -257,10 +197,7 @@ class FunkinSprite extends FlxSprite
 		if (animationIsNull)
 			return;
 
-		if (atlas != null)
-			atlas.anim.curFrame = atlas.anim.length - 1;
-		else
-			animation.curAnim.finish();
+		animation.curAnim.finish();
 	}
 
 	/**
@@ -273,28 +210,18 @@ class FunkinSprite extends FlxSprite
 		if (animationIsNull)
 			return false;
 
-		return ((atlas != null) ? atlas.anim.isPlaying : animation.curAnim.paused) ?? false;
+		return animation?.curAnim?.paused ?? false;
 	}
 
 	function set_animPaused(value:Bool):Bool
 	{
 		if (animationIsNull)
 			return value;
 
-		if (atlas != null)
-		{
-			if (value)
-				atlas.anim.pause();
-			else
-				atlas.anim.resume();
-		}
+		if (value)
+			animation.curAnim.pause();
 		else
-		{
-			if (value)
-				animation.curAnim.pause();
-			else
-				animation.curAnim.resume();
-		}
+			animation.curAnim.resume();
 
 		return value;
 	}
@@ -307,10 +234,7 @@ class FunkinSprite extends FlxSprite
 	@:privateAccess
 	public function animationExists(name:String):Bool
 	{
-		if (atlas != null)
-			return atlas.anim.symbolDictionary.get(name) != null;
-		else
-			return animation?.exists(name) ?? false;
+		return animation?.exists(name) ?? false;
 	}
 
 	/**
@@ -326,20 +250,10 @@ class FunkinSprite extends FlxSprite
 		{
 			_onAnimFinished = new FlxTypedSignal<String->Void>();
 
-			if (atlas != null)
-			{
-				atlas.anim.onComplete.add(() ->
-				{
-					_onAnimFinished.dispatch(currentAnim);
-				});
-			}
-			else
+			animation.onFinish.add((_) ->
 			{
-				animation.onFinish.add((_) ->
-				{
-					_onAnimFinished.dispatch(currentAnim);
-				});
-			}
+				_onAnimFinished.dispatch(currentAnim);
+			});
 		}
 
 		return _onAnimFinished;
@@ -354,65 +268,11 @@ class FunkinSprite extends FlxSprite
 		if (animationIsNull)
 			return false;
 
-		if (atlas != null)
-		{
-			var animData = atlas.anim.symbolDictionary.get(id);
-			if (animData == null)
-				return false;
-			return animData.length > 1;
-		}
-		else
-		{
-			var animData = animation.getByName(id);
-			if (animData == null)
-				return false;
-			return animData.numFrames > 1;
-		}
-	}
-
-	@:noCompletion
-	override function set_width(value:Float):Float
-	{
-		if (atlas != null)
-		{
-			atlas.width = value;
-			return atlas.width;
-		}
-
-		return super.set_width(value);
-	}
-
-	@:noCompletion
-	override function get_width():Float
-	{
-		if (atlas != null)
-		{
-			return atlas.width;
-		}
-
-		return super.get_width();
-	}
-
-	@:noCompletion
-	override function set_height(value:Float):Float
-	{
-		if (atlas != null)
-		{
-			atlas.height = value;
-			return atlas.height;
-		}
-
-		return super.set_height(value);
-	}
-
-	override function get_height():Float
-	{
-		if (atlas != null)
-		{
-			return atlas.height;
-		}
+		var animData = animation.getByName(id);
+		if (animData == null)
+			return false;
 
-		return super.get_height();
+		return animData.numFrames > 1;
 	}
 }