Provide the readInt8 function call in plain_dictionary codec with arguments

[kevincfz]

In codec/plain_dictionary.js, we are calling: readInt8() without argument:

opts.bitWidth = cursor.buffer.slice(cursor.offset, cursor.offset+1).readInt8();

This is fine for node.js, because its readInt8()method has a default argument of offset = 0. See https://github.com/nodejs/node/blob/master/lib/internal/buffer.js#L411.

But when using this package in browser, it breaks because the buffer method (provided by shim packages) does not have such a default argument. For example: https://github.com/feross/buffer/blob/master/index.js#L1221

Accepting this change will make this package compatible to use in browser.

[ZJONSSON]

Excellent @FZSS thank you!

[kevincfz]

appreciate the prompt response! If you haven’t done so, can you consider publishing an updated version to npm?

[ZJONSSON]

published as + parquetjs-lite@0.8.7
Keep in mind that the branch published is parquetjs-lite