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

Accept no digits in the fractional part #302

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kou
Copy link
Member

@kou kou commented Sep 5, 2024

fix #301

For example, "0.E-9" and "0." are accepted.

FYI:

Float()/.to_f will accept "0.E-9":
https://bugs.ruby-lang.org/issues/20705

The current Float doesn't accept "0." but .to_f accepts "0.".

fix ruby#301

For example, "0.E-9" and "0." are accepted.

FYI:

`Float()`/`.to_f` will accept "0.E-9":
https://bugs.ruby-lang.org/issues/20705

The current `Float` doesn't accept "0." but `.to_f` accepts "0.".
@kou kou requested a review from mrkn September 6, 2024 02:20
@kou
Copy link
Member Author

kou commented Oct 3, 2024

@mrkn What do you think about this?

@mrkn
Copy link
Member

mrkn commented Oct 3, 2024

@kou I'm in favor of this change, but I want to merge this after Ruby's [Feature #20705] is closed.

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.

Cannot parse '0.E-9'
2 participants