From: Feross Aboukhadijeh <feross@feross.org>
Date: Wed, 4 Dec 2013 10:14:39 +0000 (-0800)
Subject: remove ArrayBuffer constructor API addition
X-Git-Url: https://zoso.dev/?a=commitdiff_plain;h=0d66c8d36b2fb545237a699f00f8febb323c8e88;p=buffer.git

remove ArrayBuffer constructor API addition

It’s more important to match node’s API exactly than to be clever or
useful
---

diff --git a/index.js b/index.js
index 7ddec77..9077ea5 100644
--- a/index.js
+++ b/index.js
@@ -22,8 +22,6 @@ var browserSupport
  */
 function Buffer (subject, encoding) {
   var type = typeof subject
-  var length
-  var buf
 
   // Work-around: node's base64 implementation
   // allows for non-padded strings while base64-js
@@ -36,6 +34,7 @@ function Buffer (subject, encoding) {
   }
 
   // Find the length
+  var length
   if (type === 'number')
     length = coerce(subject)
   else if (type === 'string')
@@ -45,14 +44,7 @@ function Buffer (subject, encoding) {
   else
     throw new Error('First argument needs to be a number, array or string.')
 
-  if (subject instanceof ArrayBuffer) {
-    // API Addition: do something reasonable with an ArrayBuffer. Use it as
-    // backing buffer for the Uint8Array.
-    buf = augment(new Uint8Array(subject))
-  } else {
-    buf = augment(new Uint8Array(length))
-  }
-
+  var buf = augment(new Uint8Array(length))
   if (Buffer.isBuffer(subject)) {
     // Speed optimization -- use set if we're copying from a Uint8Array
     buf.set(subject)