double accessor not numeric-friendly like int/long accessors about klaxon

[cbeust]

If this is valid JSON, sure.

…

-- Cédric

On Mon, Sep 4, 2017 at 8:35 PM, Julian ***@***.***> wrote: I have a need for a floating point field in a JSON object. In my mind, '0' is a valid double as well, and certainly may be returned from my backend along with '1.2345'. However, JsonObject::double does not play nicely with other Number types like JsonObject::long for some reason. Does it make sense to change this? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#66>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAFoojkxMvPqJ3mY6W58fP2pKef-ozBgks5sfMFtgaJpZM4PMZ1o> .

[CapnSpellcheck]

This seems to be a workaround: (json["lat"] as Number).toDouble()

[cbeust]

Indeed. Ok to close?

[CapnSpellcheck]

It could be a stumbling block for people, especially since it could crash an unwary application.

…

On Mon, Sep 4, 2017 at 10:51 PM, Cedric Beust ***@***.***> wrote: Indeed. Ok to close? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#66 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAfQEEQfWKqdLUYCXLhNam3HtuzOwk_Fks5sfMU1gaJpZM4PMZ1o> .

[fboldog]

@CapnSpellcheck i wrote a little extension for Klaxon long time ago: https://github.com/fboldog/ext4klaxon
which is support this.

[CapnSpellcheck]

Why don't you have a floating point?? 👎

I need to support an int converted to float or double, not the other way around.