Skip to content

Conversation

@jerethk
Copy link
Contributor

@jerethk jerethk commented Oct 8, 2025

Main thing is to fix the bug that breaks his thrust/flying.
A line was erroneously added in commit 6ab2d35, setting the thrustPitch to ONE_16 (1440 degrees), which then causes the thrust to oscillate wildly with a sinusoidal pattern.

Took the opportunity to refactor the code for readability, add more enums, fix some variable names, add more comments

- add comments
- use enum for timing values
- rename "accel" to "lateralAccel"
This partially undoes a change from commit 28a91f3
The result is the same, but the refactored code is easier to reason through and debug
Line 341 is certainly a mistake, the lower number must come first when clamping

Line 462 was added by 6ab2d35 and breaks Boba Fett's thrust behaviour by setting the pitch to an extremely high number (1440 degrees)
The interval for switching directions in figure8 mode should have its own enum. It should not depend on whether the facing-player fix is enabled.

Set the shoot delay to 291 ticks regardless of whether the facing-player fix is enabled. If the delay is halved, he shoots more frequently and becomes very hard to fight.

In both cases, we are sticking with the DOS values
yAccel/thrustMax should be thrustFactor (a value that the thrustScale is multiplied by)
thrustPitch should be an ANGLE14_32 rather than FIXED16_16
@luciusDXL luciusDXL merged commit 5696494 into luciusDXL:master Jan 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants