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

Saner behavior in rb_gsl_complex_printf. #50

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Saner behavior in rb_gsl_complex_printf. #50

wants to merge 2 commits into from

Commits on Sep 23, 2017

  1. Saner behavior in rb_gsl_complex_printf. 

    Use Ruby string formatting instead of sprintf in rb_gsl_complex_printf.
This changes the behavior in some ways:
 * the function will no longer print/leak garbage from the stack if it is given a format string with too many directives. It will raise an ArgumentError instead.
 * the function can no longer crash ruby by overflowing the stack with a long format string
 * the availability and behavior of directives will match ruby sprintf and not given system's libc
 * saner type conversion rules will be used (e.g. "%d" will actually print out the integer parts of the complex number instead of reinterpreting the bits of the float as an integer)
    Jakub Zakrzewski committed Sep 23, 2017
    Configuration menu
    Copy the full SHA
    8420c5a View commit details
    Browse the repository at this point in the history

  2. Ruby 1.9 compatibility fix

    Jakub Zakrzewski committed Sep 23, 2017
    Configuration menu
    Copy the full SHA
    ce0aeac View commit details
    Browse the repository at this point in the history