Consider Node.js native crc32 method instead of buffer-crc32 dependency

thejoshwolfe / yazl

yet another zip library for node

MIT License

342 stars 45 forks source link

Consider Node.js native crc32 method instead of buffer-crc32 dependency #82

Open ijisol opened 1 month ago

ijisol commented 1 month ago

zlib.crc32(data[, value]) is added in v22.2.0 and v20.15.0.

As a side note, if you drop support for older Node.js versions, there's no reason to use the wrapper functions for bufferFrom() and bufferIncludes() anymore.

thejoshwolfe commented 1 month ago

thanks for the heads up about the native crc32 implementation! that would be much better when available. I'll take a look.