mcollina / msgpack5

A msgpack v5 implementation for node.js, with extension points / msgpack.org[Node]
MIT License
493 stars 76 forks source link

support for map32 #69

Closed imnotjames closed 6 years ago

imnotjames commented 6 years ago

This adds support and test for map32 - a map with items of up to length 2^32-1

A few tests were added to check that the support is working as expected.

Fixes #54

imnotjames commented 6 years ago

I'm not sure why the message map too big to decode in JS was in there as an error.

I can't find where this limitation exists in any browser or node version, so I'm opening this as a pull request to kick off the CI.

If you can point me in a case where this is a problem I'd be happy to try to work around the issues. I only see where node has slowdowns with huge payloads.

mcollina commented 6 years ago

Good spot! Thanks!