Skip to content

Conversation

@SeiferTim
Copy link

Should use subtract and not add for dy.

Should use subtract and not add for dy.
@Dovyski
Copy link
Member

Dovyski commented Dec 28, 2012

That's strange, maybe some charset change.

About the fix itself, have you checked if this change does not affected any other parts of Flixel? I've read in issue #53 that FlxU.getAngle() uses it and expects the "old and wrong" values.

@moly
Copy link
Member

moly commented Jan 11, 2013

This fix breaks path following, objects will now travel away from the next node instead of towards it. FlxU.rotatePoint is used here: https://github.com/FlixelCommunity/flixel/blob/dev/org/flixel/FlxObject.as#L662

I suspect the problem is that FlxU.getAngle contains a similar error.

@Dovyski
Copy link
Member

Dovyski commented Apr 9, 2013

Are you still working on this fix, @SeiferTim ?

@moly moly mentioned this pull request May 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants