search

khoarus / rapidjson

Automatically exported from code.google.com/p/rapidjson
MIT License
0 stars 0 forks source link

GenericValue type constructor for Number does not set type #57

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. rapidjson::Value num(42);
2. rapidjson::Value v(num.GetType());
3. std::cout << v.GetType()

What is the expected output? What do you see instead?
Expected: 6(kNumberType)
Seen: 0(kNullType)

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

Please provide any additional information below.
This is a real bummer :( btw nicely written library

Original issue reported on code.google.com by austindo...@gmail.com on 23 Jan 2013 at 2:22

GoogleCodeExporter commented 8 years ago 
I believe the solution for this is along the lines of,

http://pastebin.com/jbhep9LH

Original comment by austindo...@gmail.com on 23 Jan 2013 at 2:27

GoogleCodeExporter commented 8 years ago

Original comment by milo...@gmail.com on 10 Apr 2013 at 5:16

GoogleCodeExporter commented 8 years ago 
When this fix will be in trunk?

Original comment by olex...@gmail.com on 7 Aug 2013 at 6:42

GoogleCodeExporter commented 8 years ago 
I have pushed a fix for this issue to my GitHub fork at
https://github.com/pah/rapidjson/commit/fe2fe062

Original comment by philipp....@gmail.com on 1 Feb 2014 at 6:54

GoogleCodeExporter commented 8 years ago 
https://github.com/miloyip/rapidjson/pull/10

Original comment by milo...@gmail.com on 25 Jun 2014 at 4:12