darrachequesne / notepack

A fast Node.js implementation of the latest MessagePack spec
MIT License
75 stars 19 forks source link

Preserve the offset and length when creating a DataView from an ArrayBuffer view #11

Closed nkochakian closed 7 years ago

nkochakian commented 7 years ago

Fixes an issue where a view passed to the browser decoder's constructor will start reading the entire underlying ArrayBuffer instead of the portion that the view references.

This also fixes darrachequesne/notepack#10.

darrachequesne commented 7 years ago

Hi! Thanks for the pull request, could you add a test please?

darrachequesne commented 7 years ago

@nkochakian nice, thanks a lot!