From 55f2cffc22e6fe1b5ea0b76a23a8e9cb94e667f8 Mon Sep 17 00:00:00 2001 From: Sebastian McKenzie Date: Wed, 4 Feb 2015 10:05:20 +1100 Subject: [PATCH] fix linting errors --- .../transformers/es6/classes.js | 2 +- .../minification/dead-code-elimination.js | 4 ++-- .../minification/remove-console-calls.js | 2 +- .../minification/rename-local-variables.js | 2 +- lib/6to5/traverse/scope.js | 23 ++++++++----------- 5 files changed, 15 insertions(+), 18 deletions(-) diff --git a/lib/6to5/transformation/transformers/es6/classes.js b/lib/6to5/transformation/transformers/es6/classes.js index 04347b09c3..2ede1861a8 100644 --- a/lib/6to5/transformation/transformers/es6/classes.js +++ b/lib/6to5/transformation/transformers/es6/classes.js @@ -169,7 +169,7 @@ Class.prototype.buildBody = function () { // we have no constructor, we have a super, and the super doesn't appear to be falsy if (!this.hasConstructor && superName && !t.isFalsyExpression(superName)) { var helperName = "class-super-constructor-call"; - if (this.isLoose) helperName += "-loose" + if (this.isLoose) helperName += "-loose"; constructor.body.body.push(util.template(helperName, { CLASS_NAME: className, SUPER_NAME: this.superName diff --git a/lib/6to5/transformation/transformers/minification/dead-code-elimination.js b/lib/6to5/transformation/transformers/minification/dead-code-elimination.js index 99f5892c32..0d92cddb4e 100644 --- a/lib/6to5/transformation/transformers/minification/dead-code-elimination.js +++ b/lib/6to5/transformation/transformers/minification/dead-code-elimination.js @@ -2,7 +2,7 @@ var t = require("../../../types"); exports.optional = true; -exports.ExpressionStatement = function (node, parent, scope, context, file) { +exports.ExpressionStatement = function (node, parent, scope, context) { // remove consequenceless expressions such as local variables and literals // // var foo = true; foo; -> var foo = true; @@ -16,7 +16,7 @@ exports.ExpressionStatement = function (node, parent, scope, context, file) { }; exports.IfStatement = { - exit: function (node, parent, scope, context, file) { + exit: function (node, parent, scope, context) { // todo: in scenarios where we can just return the consequent or // alternate we should drop the block statement if it contains no // block scoped variables diff --git a/lib/6to5/transformation/transformers/minification/remove-console-calls.js b/lib/6to5/transformation/transformers/minification/remove-console-calls.js index 950d9dcdf6..09ff326e15 100644 --- a/lib/6to5/transformation/transformers/minification/remove-console-calls.js +++ b/lib/6to5/transformation/transformers/minification/remove-console-calls.js @@ -6,7 +6,7 @@ var isConsole = t.buildMatchMemberExpression("console", true); exports.optional = true; -exports.CallExpression = function (node, parent, scope, context, file) { +exports.CallExpression = function (node, parent, scope, context) { if (isConsole(node.callee)) { context.remove(); } diff --git a/lib/6to5/transformation/transformers/minification/rename-local-variables.js b/lib/6to5/transformation/transformers/minification/rename-local-variables.js index 3090d437e1..56a8a43699 100644 --- a/lib/6to5/transformation/transformers/minification/rename-local-variables.js +++ b/lib/6to5/transformation/transformers/minification/rename-local-variables.js @@ -1,4 +1,4 @@ -var t = require("../../../types"); +//var t = require("../../../types"); exports.optional = true; diff --git a/lib/6to5/traverse/scope.js b/lib/6to5/traverse/scope.js index 47dc2f28ab..ca04afd3b3 100644 --- a/lib/6to5/traverse/scope.js +++ b/lib/6to5/traverse/scope.js @@ -2,19 +2,16 @@ module.exports = Scope; -var isBoolean = require("lodash/lang/isBoolean"); -var isNumber = require("lodash/lang/isNumber"); -var contains = require("lodash/collection/contains"); -var isString = require("lodash/lang/isString"); -var traverse = require("./index"); -var defaults = require("lodash/object/defaults"); -var globals = require("globals"); -var flatten = require("lodash/array/flatten"); -var extend = require("lodash/object/extend"); -var object = require("../helpers/object"); -var each = require("lodash/collection/each"); -var has = require("lodash/object/has"); -var t = require("../types"); +var contains = require("lodash/collection/contains"); +var traverse = require("./index"); +var defaults = require("lodash/object/defaults"); +var globals = require("globals"); +var flatten = require("lodash/array/flatten"); +var extend = require("lodash/object/extend"); +var object = require("../helpers/object"); +var each = require("lodash/collection/each"); +var has = require("lodash/object/has"); +var t = require("../types"); /** * This searches the current "scope" and collects all references/bindings