qbektrix / xml2json-xslt

Automatically exported from code.google.com/p/xml2json-xslt
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

JSON doesn't allow octal numbers. #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Douglas Crockford has pointed out that JSON doesn't allow octal numbers.

The octal test cases and code should be removed.

Ciao!

Original issue reported on code.google.com by docw...@gmail.com on 31 Mar 2008 at 5:30

GoogleCodeExporter commented 9 years ago
See http://json.org/ for more information.
Ditto for hex (should add a test for that).

I think the proper handling will be to convert oct and hex to base10.

Original comment by docw...@gmail.com on 31 Mar 2008 at 5:32

GoogleCodeExporter commented 9 years ago
Hmm.... I cannot find any octal or hexadecimal converters in oct or hex.  While 
I could write one (and it'd 
probably be error prone) I think I'll punt unless someone wants one badly.   
The only person asking about one 
was root.node in Issue 2.  If he/she is interested enough to whip up a way to 
convert them reliably into decimal, 
I'll take another look at it.

Fixed in r28

Ciao!

Original comment by docw...@gmail.com on 31 Mar 2008 at 8:27