search

kig / DataStream.js

DataStream.js is a library for reading data from ArrayBuffers
173 stars 39 forks source link

Small Fixes #16

Closed r0l1 closed 8 years ago

r0l1 commented 8 years ago

Just some small fixes ;)

kig commented 8 years ago

Can you check if the unit test thing passes with these changes? You should probably change "=== null" to "=== undefined" if you want to use the === comparator. Perf-wise not expecting much of a win from the === comparisons, but I like that the === is clearer in intent.

r0l1 commented 8 years ago

Thanks for having a look into it. However I won't continue to work with DataStreams (-> https://github.com/desertbit/byte-buffer). Feel free to merge or reject.

kig commented 8 years ago

Ooh, that looks neat