From 298cfac7f4429eacf7c0867fadc9dc70a7b1e6e3 Mon Sep 17 00:00:00 2001 From: Sebastian McKenzie Date: Thu, 12 Feb 2015 21:55:31 +1100 Subject: [PATCH] fix linting errors --- lib/6to5/messages.js | 2 +- lib/6to5/transformation/transformers/es6/classes.js | 6 ++++-- .../transformers/minification/rename-local-variables.js | 2 +- .../transformers/validation/undeclared-variable-check.js | 2 -- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/6to5/messages.js b/lib/6to5/messages.js index dd55873d0f..21d4ad4eec 100644 --- a/lib/6to5/messages.js +++ b/lib/6to5/messages.js @@ -1,4 +1,4 @@ -var toArray = require("lodash/lang/toArray"); +var util = require("util"); exports.messages = { tailCallReassignmentDeopt: "Function reference has been reassigned so it's probably be dereferenced so we can't optimise this with confidence", diff --git a/lib/6to5/transformation/transformers/es6/classes.js b/lib/6to5/transformation/transformers/es6/classes.js index c2c42a36a8..5cdf30ceea 100644 --- a/lib/6to5/transformation/transformers/es6/classes.js +++ b/lib/6to5/transformation/transformers/es6/classes.js @@ -257,13 +257,15 @@ ClassTransformer.prototype.pushMethod = function (node) { ClassTransformer.prototype.pushProperty = function (node) { if (!node.value) return; + var key; + if (node.static) { - var key = t.memberExpression(this.className, node.key); + key = t.memberExpression(this.className, node.key); this.body.push( t.expressionStatement(t.assignmentExpression("=", key, node.value)) ); } else { - var key = t.memberExpression(t.thisExpression(), node.key); + key = t.memberExpression(t.thisExpression(), node.key); this.constructor.body.body.unshift( t.expressionStatement(t.assignmentExpression("=", key, node.value)) ); diff --git a/lib/6to5/transformation/transformers/minification/rename-local-variables.js b/lib/6to5/transformation/transformers/minification/rename-local-variables.js index 68f8c781ae..1617a84bd0 100644 --- a/lib/6to5/transformation/transformers/minification/rename-local-variables.js +++ b/lib/6to5/transformation/transformers/minification/rename-local-variables.js @@ -2,7 +2,7 @@ exports.optional = true; -exports.Scopable = function (node, parent, scope) { +exports.Scopable = function () { //for (var name in scope.bindings) { // scope.rename(name, scope.generateUidIdentifier("a").name); //} diff --git a/lib/6to5/transformation/transformers/validation/undeclared-variable-check.js b/lib/6to5/transformation/transformers/validation/undeclared-variable-check.js index b02a7354ba..3bea68ad80 100644 --- a/lib/6to5/transformation/transformers/validation/undeclared-variable-check.js +++ b/lib/6to5/transformation/transformers/validation/undeclared-variable-check.js @@ -10,8 +10,6 @@ exports.Identifier = function (node, parent, scope, file) { if (!t.isReferenced(node, parent)) return; if (scope.hasBinding(node.name)) return; - var msg = "Reference to undeclared variable"; - // get the closest declaration to offer as a suggestion // the variable name may have just been mistyped