Question about handling of NaN

mcollina / msgpack5

A msgpack v5 implementation for node.js, with extension points / msgpack.org[Node]

MIT License

493 stars 76 forks source link

Question about handling of NaN #74

Closed valichek closed 6 years ago

valichek commented 6 years ago

I'm using this lib in browser and noticed that NaN is encoded to float32 by default. Would expect it to raise encoding error, same as undefined.

mcollina commented 6 years ago

Possibly is a bug! Would you like to send a PR?

valichek commented 6 years ago

@mcollina yes, going to prepare it soon