diff options
-rw-r--r-- | lib/.eslintrc | 1 | ||||
-rw-r--r-- | src/.eslintrc | 3 | ||||
-rw-r--r-- | tools/eslint-rules/buffer-constructor.js | 25 |
3 files changed, 29 insertions, 0 deletions
diff --git a/lib/.eslintrc b/lib/.eslintrc index 341e9327c7..8fb98b7c18 100644 --- a/lib/.eslintrc +++ b/lib/.eslintrc @@ -1,3 +1,4 @@ rules: # Custom rules in tools/eslint-rules require-buffer: 2 + buffer-constructor: 2 diff --git a/src/.eslintrc b/src/.eslintrc new file mode 100644 index 0000000000..baf9c809c0 --- /dev/null +++ b/src/.eslintrc @@ -0,0 +1,3 @@ +rules: + # Custom rules in tools/eslint-rules + buffer-constructor: 2 diff --git a/tools/eslint-rules/buffer-constructor.js b/tools/eslint-rules/buffer-constructor.js new file mode 100644 index 0000000000..938598e8db --- /dev/null +++ b/tools/eslint-rules/buffer-constructor.js @@ -0,0 +1,25 @@ +/** + * @fileoverview Require use of new Buffer constructor methods in lib + * @author James M Snell + */ +'use strict'; + +//------------------------------------------------------------------------------ +// Rule Definition +//------------------------------------------------------------------------------ +const msg = 'Use of the Buffer() constructor has been deprecated. ' + + 'Please use either Buffer.alloc(), Buffer.allocUnsafe(), ' + + 'or Buffer.from()'; + +function test(context, node) { + if (node.callee.name === 'Buffer') { + context.report(node, msg); + } +} + +module.exports = function(context) { + return { + 'NewExpression': (node) => test(context, node), + 'CallExpression': (node) => test(context, node) + }; +}; |