boonproject / boon

Simple opinionated Java for the novice to expert level Java Programmer. Low Ceremony. High Productivity.
http://richardhightower.github.io/site/Boon/Welcome.html
Apache License 2.0
522 stars 102 forks source link

Cannot parse double field such as {"doubleValue":"NaN"} #311

Closed michael-g closed 9 years ago

michael-g commented 9 years ago

Exception is raised from line 1131 in CharScanner.java in method parseDouble(char[],int,int).

RichardHightower commented 9 years ago

Ok.. that sounds like an easy fix. Thanks.

On Thu, Apr 16, 2015 at 1:35 PM, Michael notifications@github.com wrote:

Exception is raised from line 1131 in CharScanner.java in method parseDouble(char[],int,int).

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311.

Rick Hightower (415) 968-9037 Profile http://www.google.com/profiles/RichardHightower

michael-g commented 9 years ago

Sweet! Sorry for the cursory bug report (no test case, etc), but I'm in trouble with the missus for spending too much time with my computer. Good fag-packet results vs Gson, btw, on large message sizes. Timed at c. 9 seconds vs 13 seconds for 1000 runs over roughly 30 separate messages. Nice one! On 16 Apr 2015 23:22, "Richard Hightower" notifications@github.com wrote:

Ok.. that sounds like an easy fix. Thanks.

On Thu, Apr 16, 2015 at 1:35 PM, Michael notifications@github.com wrote:

Exception is raised from line 1131 in CharScanner.java in method parseDouble(char[],int,int).

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311.

Rick Hightower (415) 968-9037 Profile http://www.google.com/profiles/RichardHightower

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311#issuecomment-93852620.

RichardHightower commented 9 years ago

https://answers.yahoo.com/question/index?qid=20080107070851AAqiHyu

I had to look that one up...

The literal translation is "back of a cigarette packet".

We don't say fag in the US. It means something else.

We say the same thing but its on a match book cover. :)

But same idea... :)

Lost in translation no more.

On Thu, Apr 16, 2015 at 3:28 PM, Michael notifications@github.com wrote:

Sweet! Sorry for the cursory bug report (no test case, etc), but I'm in trouble with the missus for spending too much time with my computer. Good fag-packet results vs Gson, btw, on large message sizes. Timed at c. 9 seconds vs 13 seconds for 1000 runs over roughly 30 separate messages. Nice one! On 16 Apr 2015 23:22, "Richard Hightower" notifications@github.com wrote:

Ok.. that sounds like an easy fix. Thanks.

On Thu, Apr 16, 2015 at 1:35 PM, Michael notifications@github.com wrote:

Exception is raised from line 1131 in CharScanner.java in method parseDouble(char[],int,int).

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311.

Rick Hightower (415) 968-9037 Profile http://www.google.com/profiles/RichardHightower

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311#issuecomment-93852620.

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311#issuecomment-93853600.

Rick Hightower (415) 968-9037 Profile http://www.google.com/profiles/RichardHightower

michael-g commented 9 years ago

LOL. Yep, that comment is giving quite a bit away in terms of clarity! Must be what happens when you try to type something on a smartphone...

RichardHightower commented 9 years ago

Ok.. looking into this now.

RichardHightower commented 9 years ago

Fixed.

michael-g commented 9 years ago

Brilliant. Will test drive soon. Thanks! On 19 Apr 2015 02:49, "Richard Hightower" notifications@github.com wrote:

Fixed.

— Reply to this email directly or view it on GitHub https://github.com/boonproject/boon/issues/311#issuecomment-94221951.