Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #19

Closed
wants to merge 3 commits into from
Closed

Fixes #19

wants to merge 3 commits into from

Conversation

akheron
Copy link
Member

@akheron akheron commented May 31, 2023

Make LMaara global. Its value stayed the same between hyppy() function calls by pure accident.

Mimic frame rate limiter more like the original DOS version does it, i.e. wait that a new frame starts before rendering instead of just waiting for any change to framecount. Also make frame counting more accurate.

Use double precision floating point arithmetic everywhere. With single precision floating points, implementation details of the compiler affect how calculations are made, e.g. when to widen the number to double precision and convert back to single decision. As such, using double precision everywhere isn't probably worse than compling the original code with Free Pascal, which might have different implementation details than Turbo Pascal that was originally used.

Because of the aforementioned compiler implementation details, using double precision is vital to be able to reproduce the precise computations in another programming language (see /~https://github.com/akheron/skijump3-remake).

akheron added 3 commits May 31, 2023 20:58
Its value persisted between hyppy() function calls by accident.
Mimic the DOS version's frame rate limiting more accurately.
This was referenced Apr 7, 2024
@veikkos
Copy link
Member

veikkos commented Apr 7, 2024

First two commits merged in #26 and #27. Precicion change deemed as too intruisive by the author after all. Closing this PR.

@veikkos veikkos closed this Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants