From 1e13c0147b7436a78b5b288313b5813860a0cc91 Mon Sep 17 00:00:00 2001
From: "P. Mike" <mihan@panfilov.biz>
Date: Sat, 29 Dec 2018 16:17:40 +0300
Subject: [PATCH 1/2] lib/buffer: fix unused variable initialization in
 SlowBuffer method lib/buffer: change var to let/const variables
 initialization

---
 lib/buffer.js | 33 ++++++++++++++++-----------------
 1 file changed, 16 insertions(+), 17 deletions(-)

diff --git a/lib/buffer.js b/lib/buffer.js
index 01d2764c6d566b..63bd5689fa159b 100644
--- a/lib/buffer.js
+++ b/lib/buffer.js
@@ -102,7 +102,7 @@ const constants = Object.defineProperties({}, {
 });
 
 Buffer.poolSize = 8 * 1024;
-var poolSize, poolOffset, allocPool;
+let poolSize, poolOffset, allocPool;
 
 setupBufferJS(Buffer.prototype, bindingObj);
 
@@ -221,7 +221,7 @@ Buffer.from = function from(value, encodingOrOffset, length) {
   if (valueOf !== null && valueOf !== undefined && valueOf !== value)
     return Buffer.from(valueOf, encodingOrOffset, length);
 
-  var b = fromObject(value);
+  const b = fromObject(value);
   if (b)
     return b;
 
@@ -307,13 +307,12 @@ Buffer.allocUnsafeSlow = function allocUnsafeSlow(size) {
 // If --zero-fill-buffers command line argument is set, a zero-filled
 // buffer is returned.
 function SlowBuffer(length) {
-  const len = +length;
-  // eslint-disable-next-line eqeqeq
-  if (len != length)
+  if (typeof length !== 'number')
+    length = +length;
+  if (isNaN(length))
     length = 0;
-  else
-    assertSize(len);
-  return createUnsafeBuffer(len);
+  assertSize(length);
+  return createUnsafeBuffer(length);
 }
 
 Object.setPrototypeOf(SlowBuffer.prototype, Uint8Array.prototype);
@@ -326,7 +325,7 @@ function allocate(size) {
   if (size < (Buffer.poolSize >>> 1)) {
     if (size > (poolSize - poolOffset))
       createPool();
-    var b = new FastBuffer(allocPool, poolOffset, size);
+    const b = new FastBuffer(allocPool, poolOffset, size);
     poolOffset += size;
     alignPool();
     return b;
@@ -335,7 +334,7 @@ function allocate(size) {
 }
 
 function fromString(string, encoding) {
-  var length;
+  let length;
   if (typeof encoding !== 'string' || encoding.length === 0) {
     if (string.length === 0)
       return new FastBuffer();
@@ -354,7 +353,7 @@ function fromString(string, encoding) {
 
   if (length > (poolSize - poolOffset))
     createPool();
-  var b = new FastBuffer(allocPool, poolOffset, length);
+  let b = new FastBuffer(allocPool, poolOffset, length);
   const actual = b.write(string, encoding);
   if (actual !== length) {
     // byteLength() may overestimate. That's a rare case, though.
@@ -456,7 +455,7 @@ Buffer.isEncoding = function isEncoding(encoding) {
 Buffer[kIsEncodingSymbol] = Buffer.isEncoding;
 
 Buffer.concat = function concat(list, length) {
-  var i;
+  let i;
   if (!Array.isArray(list)) {
     throw new ERR_INVALID_ARG_TYPE(
       'list', ['Array', 'Buffer', 'Uint8Array'], list);
@@ -473,10 +472,10 @@ Buffer.concat = function concat(list, length) {
     length = length >>> 0;
   }
 
-  var buffer = Buffer.allocUnsafe(length);
-  var pos = 0;
+  const buffer = Buffer.allocUnsafe(length);
+  let pos = 0;
   for (i = 0; i < list.length; i++) {
-    var buf = list[i];
+    const buf = list[i];
     if (!isUint8Array(buf)) {
       // TODO(BridgeAR): This should not be of type ERR_INVALID_ARG_TYPE.
       // Instead, find the proper error code for this.
@@ -791,7 +790,7 @@ function bidirectionalIndexOf(buffer, val, byteOffset, encoding, dir) {
 }
 
 function slowIndexOf(buffer, val, byteOffset, encoding, dir) {
-  var loweredCase = false;
+  let loweredCase = false;
   for (;;) {
     switch (encoding) {
       case 'utf8':
@@ -926,7 +925,7 @@ Buffer.prototype.write = function write(string, offset, length, encoding) {
       length = undefined;
     }
 
-    var remaining = this.length - offset;
+    const remaining = this.length - offset;
     if (length === undefined || length > remaining)
       length = remaining;
 

From fde0b07626c2591768096d09e553eee8b6cd87fb Mon Sep 17 00:00:00 2001
From: "P. Mike" <mihan@panfilov.biz>
Date: Sun, 30 Dec 2018 04:37:25 +0300
Subject: [PATCH 2/2] lib/buffer: remove isNaN checking

---
 lib/buffer.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/lib/buffer.js b/lib/buffer.js
index 63bd5689fa159b..0b765832663b3a 100644
--- a/lib/buffer.js
+++ b/lib/buffer.js
@@ -309,8 +309,6 @@ Buffer.allocUnsafeSlow = function allocUnsafeSlow(size) {
 function SlowBuffer(length) {
   if (typeof length !== 'number')
     length = +length;
-  if (isNaN(length))
-    length = 0;
   assertSize(length);
   return createUnsafeBuffer(length);
 }