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

Patch: use BigDecimal for numbers instead of Double #73

Open
GoogleCodeExporter opened this issue Dec 24, 2015 · 4 comments
Open

Patch: use BigDecimal for numbers instead of Double #73

GoogleCodeExporter opened this issue Dec 24, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. put really large decimal number with also small parts in json input
2. parse
3. encode
4. detail is lost and/or formatting changes to scientific notation (which is 
not exactly a defect)

What is the expected output? What do you see instead?

I'd like the numberical value to be preserved, with precision information if 
possible 

What version of the product are you using? On what operating system?

1.1.1 java6 and 7

Please provide any additional information below.

Patch is simple as one only need to replace Double handling with
yylex.java:608
{ BigDecimal val=new BigDecimal(yytext()); return new 
Yytoken(Yytoken.TYPE_VALUE, val);

Original issue reported on code.google.com by [email protected] on 2 Jun 2012 at 11:02

@GoogleCodeExporter
Copy link
Author

Original comment by [email protected] on 4 Jul 2012 at 5:49

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Original comment by [email protected] on 4 Jul 2012 at 5:50

  • Added labels: Type-Enhancement
  • Removed labels: Type-Defect

@GoogleCodeExporter
Copy link
Author

Issue 57 has been merged into this issue.

Original comment by [email protected] on 10 Aug 2013 at 5:01

@GoogleCodeExporter
Copy link
Author

still not resolved 

Original comment by [email protected] on 22 Jun 2015 at 6:18

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

No branches or pull requests

1 participant