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

A more robust parse method #21

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

A more robust parse method #21

wants to merge 7 commits into from

Commits on Jul 16, 2018

  1. Add tests to test_decimal.jl

    @lcontento
    lcontento committed Jul 16, 2018
    Configuration menu
    Copy the full SHA
    1da4825 View commit details
    Browse the repository at this point in the history

  2. Rewrite parse function 

    Cases such as 1.0000001e6 are now handled correctly.
scinote and parameters functions are no longer used.
    @lcontento
    lcontento committed Jul 16, 2018
    Configuration menu
    Copy the full SHA
    935e91c View commit details
    Browse the repository at this point in the history

  3. Fix for parse(Decimal, "0.0")

    @lcontento
    lcontento committed Jul 16, 2018
    Configuration menu
    Copy the full SHA
    50f8b29 View commit details
    Browse the repository at this point in the history

  4. Add tests for leading zeros in parsed strings

    @lcontento
    lcontento committed Jul 16, 2018
    Configuration menu
    Copy the full SHA
    914a303 View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2018

  1. Changes to parse 

    * More suggestive variable names
* Allow 'E' as a exponent character
* Raise an error if '.' or 'e' appear twice in the string to be parsed
    @lcontento
    lcontento committed Nov 6, 2018
    Configuration menu
    Copy the full SHA
    bc30349 View commit details
    Browse the repository at this point in the history

  2. Raise ArgumentError instead of AssertionError

    @lcontento
    lcontento committed Nov 6, 2018
    Configuration menu
    Copy the full SHA
    c8255da View commit details
    Browse the repository at this point in the history

  3. Test for repeating e/. characters

    @lcontento
    lcontento committed Nov 6, 2018
    Configuration menu
    Copy the full SHA
    d0fb5c2 View commit details
    Browse the repository at this point in the history