[Pkg-javascript-commits] [node-browser-pack] 138/141: use `new` to instantiate all buffers (#75)

[Bastien Roucariès]

This is an automated email from the git hooks/post-receive script.

rouca pushed a commit to branch master
in repository node-browser-pack.

commit c91b8166ae21441e0d855dcc791f90f7998d07e2
Author: Myles Borins <myles.borins at gmail.com>
Date:   Wed Nov 2 15:54:43 2016 -0400

    use `new` to instantiate all buffers (#75)
    
    Using Buffer without `new` will soon stop working. It will throw
    A deprecation warning in Node v7.
---
 index.js | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/index.js b/index.js
index 9d3ab70..3b96de8 100644
--- a/index.js
+++ b/index.js
@@ -44,13 +44,13 @@ module.exports = function (opts) {
     function write (row, enc, next) {
         if (first && opts.standalone) {
             var pre = umd.prelude(opts.standalone).trim();
-            stream.push(Buffer(pre + 'return '));
+            stream.push(new Buffer(pre + 'return '));
         }
         else if (first && stream.hasExports) {
             var pre = opts.externalRequireName || 'require';
-            stream.push(Buffer(pre + '='));
+            stream.push(new Buffer(pre + '='));
         }
-        if (first) stream.push(Buffer(prelude + '({'));
+        if (first) stream.push(new Buffer(prelude + '({'));
         
         if (row.sourceFile && !row.nomap) {
             if (!sourcemap) {
@@ -81,7 +81,7 @@ module.exports = function (opts) {
             ']'
         ].join('');
 
-        stream.push(Buffer(wrappedSource));
+        stream.push(new Buffer(wrappedSource));
         lineno += newlinesIn(wrappedSource);
         
         first = false;
@@ -93,13 +93,13 @@ module.exports = function (opts) {
     }
     
     function end () {
-        if (first) stream.push(Buffer(prelude + '({'));
+        if (first) stream.push(new Buffer(prelude + '({'));
         entries = entries.filter(function (x) { return x !== undefined });
         
-        stream.push(Buffer('},{},' + JSON.stringify(entries) + ')'));
+        stream.push(new Buffer('},{},' + JSON.stringify(entries) + ')'));
 
         if (opts.standalone && !first) {
-            stream.push(Buffer(
+            stream.push(new Buffer(
                 '(' + JSON.stringify(stream.standaloneModule) + ')'
                 + umd.postlude(opts.standalone)
             ));
@@ -112,9 +112,9 @@ module.exports = function (opts) {
                     /^\/\/#/, function () { return opts.sourceMapPrefix }
                 )
             }
-            stream.push(Buffer('\n' + comment + '\n'));
+            stream.push(new Buffer('\n' + comment + '\n'));
         }
-        if (!sourcemap && !opts.standalone) stream.push(Buffer(';\n'));
+        if (!sourcemap && !opts.standalone) stream.push(new Buffer(';\n'));
 
         stream.push(null);
     }

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/node-browser-pack.git