search

cruise-automation / rosbag.js

ROS bag file reader for JavaScript 👜
Apache License 2.0
189 stars 43 forks source link

Export MessageWriter #78

Closed hhsaez closed 4 years ago

hhsaez commented 4 years ago

In this PR I'm adding MessageWriter to the exported entities so we can use it in external projects. In addition, I'm using Buffer.allocUnsafe instead of new Buffer as suggested by @MatthewSteel to prevent warnings.

Testing: all existing tests must pass

janpaul123 commented 4 years ago

(I'd bump the minor version since it's an API change; patch is just for bugfixes; but also it doesn't matter much 😄 🤷 )

davidswinegar commented 4 years ago

Merged this so it can go in with #76