Replace `new Buffer()` with `Buffer.alloc()` as it is safer. #3

Eraz1997 · 2021-10-14T13:14:42Z

Newer versions of NodeJS deprecated the Buffer constructor as it is unsafe (source: https://nodejs.org/en/docs/guides/buffer-constructor-deprecation/). These code changes might not work on Node.js ≤ 4.4.x and 5.0.0 — 5.9.x, but this should not be a problem since projects running those versions of NodeJS should use ref-array instead of ref-array-napi.

Eraz1997 · 2021-10-14T13:42:27Z

Checks on a couple of Windows platforms failed, but it seems to be a node-gyp error when building tests, not the tests failing

Replace new Buffer() with Buffer.alloc() as it is safer.

f6a0dc3

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Replace `new Buffer()` with `Buffer.alloc()` as it is safer. #3

Replace `new Buffer()` with `Buffer.alloc()` as it is safer. #3

Eraz1997 commented Oct 14, 2021

Eraz1997 commented Oct 14, 2021 •

edited

Loading

Replace new Buffer() with Buffer.alloc() as it is safer. #3

Are you sure you want to change the base?

Replace new Buffer() with Buffer.alloc() as it is safer. #3

Conversation

Eraz1997 commented Oct 14, 2021

Eraz1997 commented Oct 14, 2021 • edited Loading

Replace `new Buffer()` with `Buffer.alloc()` as it is safer. #3

Replace `new Buffer()` with `Buffer.alloc()` as it is safer. #3

Eraz1997 commented Oct 14, 2021 •

edited

Loading