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.

Sign up for GitHub

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

Jump to bottom

remove snprintf/strncat/strlen from fp_ident #33

Merged
merged 3 commits into from
Oct 16, 2024
Merged

remove snprintf/strncat/strlen from fp_ident #33

merged 3 commits into from
Oct 16, 2024
+85 −10

Commits on Sep 21, 2024

  1. remove snprintf/strncat/strlen from fp_ident 

    fp_ident is the only place tomsfast math uses string manipulation
functions when built as a library. This commit replaces them with
safe, trivial functions for concatonating static strings and numbers
together.
    @ryancdotorg @sjaeckel
    ryancdotorg authored and sjaeckel committed Sep 21, 2024
    Configuration menu
    Copy the full SHA
    537ba3e View commit details
    Browse the repository at this point in the history

  2. Add make target for standalone ident 

    Signed-off-by: Steffen Jaeckel <[email protected]>
    @sjaeckel
    sjaeckel committed Sep 21, 2024
    Configuration menu
    Copy the full SHA
    c1cc1c3 View commit details
    Browse the repository at this point in the history

  3. Slightly improve fp_ident 

    * do error checking.
* also print `SIZEOF_FP_DIGIT` and `DIGIT_SHIFT`.
* cut static buffer in half, currently the output string is 183 chars
  on my machine, so it's still more than enough for all the other options.

Signed-off-by: Steffen Jaeckel <[email protected]>
    @sjaeckel
    sjaeckel committed Sep 21, 2024
    Configuration menu
    Copy the full SHA
    1519aa1 View commit details
    Browse the repository at this point in the history