Skip to content

Use stringstream to convert real numbers from strings #293

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
Closed
cinemast opened this issue Jun 17, 2015 · 0 comments · Fixed by #294
Closed

Use stringstream to convert real numbers from strings #293

cinemast opened this issue Jun 17, 2015 · 0 comments · Fixed by #294

Comments

@cinemast
Copy link
Contributor

Hi!

Could you take a look at the following patch:
http://anonscm.debian.org/cgit/collab-maint/libjsoncpp.git/tree/debian/patches/fix-double-parsing.patch?id=7d2b3086f3cfc770a877b7dfc4f532b7fc754c15

I think it should also be imported upstream.

The problem is that in some cases "," is used instead of "." for real numbers.
It affects versions 0.10.x
cdunn2001 added a commit to cdunn2001/jsoncpp that referenced this issue Jun 19, 2015
@cdunn2001
fix ,/. problem in reader
6416350 
fixes open-source-parsers#293
@cdunn2001 cdunn2001 mentioned this issue Jun 19, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix ,/. problem in reader cdunn2001/jsoncpp
1 participant
@cinemast