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

Fix the incorrect double compare with e,E+,e+ #77

Merged
merged 1 commit into from
Dec 16, 2021

Conversation

shoothzj
Copy link
Collaborator

The use extra CPU to check if the result can be return as double without precision lost does not consider the case of double can appear e E+ e+, but the double toString only shows E.
Which will lead to an incorrect compare failure.

@shoothzj
Copy link
Collaborator Author

shoothzj commented Sep 3, 2021

@UrielCh PTAL

@caimo
Copy link

caimo commented Nov 17, 2021

I have the same problem, can you help to check this PR @erav ?

@shoothzj
Copy link
Collaborator Author

@UrielCh Could you please take a look, thank you

@UrielCh
Copy link
Contributor

UrielCh commented Dec 16, 2021

looks good...

@UrielCh UrielCh merged commit e1d26f5 into netplex:master Dec 16, 2021
@UrielCh
Copy link
Contributor

UrielCh commented Dec 16, 2021

tell me if you want a new release.

@shoothzj
Copy link
Collaborator Author

tell me if you want a new release.

Yes, I want a new release. If you have time, please release a new version. Thank you :)

@tracy-java
Copy link

tell me if you want a new release.

Yes, I want a new release too. If you have time, please release a new version. Thanks, that was helpful.

@shoothzj
Copy link
Collaborator Author

shoothzj commented Dec 28, 2021

tell me if you want a new release.

@UrielCh Happy Holidays! looking forward to your release :)

@bigDouble46
Copy link

tell me if you want a new release.

I want a new release too. Thanks.

@Lv000001
Copy link

tell me if you want a new release.

Yes, I want a new release.

@tracy-java
Copy link

@UrielCh looking forward to your release :). thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants