Improve fixed-point documentation #1455
Merged
+65
−34
Commits on Aug 8, 2024
-
-
-
Fix formatting of trig func list
There was a space between the opening parens an `SIN`
-
Simplify sine table example a bit
Use fewer magic constants, and a FOR loop. Also use a max input bound that isn't 1.0, to avoid confusion with the `1.0` that's used to shift the output range.
-
Remove misleading equations describing
DIV,MUL, andFMODThey were not equivalent, and besides, it seems more likely that the shifts would confuse newbies while not bringing useful info to experts.
-
Use more maths to describe the fixed-point functions
This should help, I think?
-
-
Co-authored-by: Sylvie <[email protected]>
-
Remove detailed explanations of some fixed-point funcs
Co-authored-by: Sylvie <[email protected]>
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.