v2.13.2

Remove unused local variable

.npmignore

@@ -10,3 +10,4 @@ dist
 tests.json
 CHANGELOG.md
 .package.json
+coverage

CHANGELOG.md

@@ -11,6 +11,14 @@
 
 _Note: Gaps between patch versions are faulty/broken releases._
 
+## 2.13.2
+
+ * **Bug Fix**
+  * Fix `super` inside of nested functions.
+ * **Internal**
+  * Move let scoping TDZ into a separate transformer until it's more solid.
+  * Use `process.stdin.write` instead of `console.log` in `bin` to avoid sprintfification.
+
 ## 2.13.1
 
  * **New Feature**

Makefile

@@ -1,6 +1,7 @@
 BROWSERIFY_CMD = node_modules/browserify/bin/cmd.js
 ISTANBUL_CMD = node_modules/istanbul/lib/cli.js cover
-UGLIFY_CMD = node_modules/uglify-js/bin/uglifyjs --mangle sort
+UGLIFY_CMD = node_modules/uglify-js/bin/uglifyjs
+#UGLIFY_CMD = node_modules/uglify-js/bin/uglifyjs --mangle sort
 JSHINT_CMD = node_modules/jshint/bin/jshint
 MOCHA_CMD = node_modules/mocha/bin/_mocha
 

NOTES.md

@@ -11,3 +11,4 @@
  * Add autoindentation.
  * Move `super` transformation from classes into a separate transformer that also supports object expressions.
  * Remove fast transformer backwards compatibility.
+ * Rename let scoping transformer to block scoping.

bin/6to5/file.js

@@ -63,7 +63,7 @@ module.exports = function (commander, filenames) {
 
       fs.writeFileSync(commander.outFile, result.code);
     } else {
-      console.log(result.code);
+      process.stdout.write(result.code + "\n");
     }
   };
 

bin/6to5/index.js

@@ -34,13 +34,10 @@ commander.on("--help", function(){
     console.log("  " + title + ":");
     console.log();
 
-    var hasOptional = true;
-
     _.each(_.keys(obj).sort(), function (key) {
       if (key[0] === "_") return;
 
       if (obj[key].optional) {
-        hasOptional = true;
         key = "[" + key + "]";
       }
 

lib/6to5/transformation/helpers/replace-supers.js

@@ -152,7 +152,7 @@ ReplaceSupers.prototype.getThisReference = function () {
     return this.topLevelThisReference;
   } else {
     var ref = this.topLevelThisReference = this.file.generateUidIdentifier("this");
-    this.methodNode.body.body.unshift(t.variableDeclaration("var", [
+    this.methodNode.value.body.body.unshift(t.variableDeclaration("var", [
       t.variableDeclarator(this.topLevelThisReference, t.thisExpression())
     ]));
     return ref;

lib/6to5/transformation/transform.js

@@ -79,6 +79,7 @@ _.each({
 
   constants:                 require("./transformers/es6-constants"),
   letScoping:                require("./transformers/es6-let-scoping"),
+  blockScopingTDZ:           require("./transformers/optional-block-scoping-tdz"),
 
   _blockHoist:               require("./transformers/_block-hoist"),
 

lib/6to5/transformation/transformers/es6-let-scoping.js

@@ -84,7 +84,7 @@ function LetScoping(loopParent, block, parent, scope, file) {
 
   this.outsideLetReferences = {};
   this.hasLetReferences     = false;
-  this.letReferences        = {};
+  this.letReferences        = block._letReferences = {};
   this.body                 = [];
 }
 
@@ -98,7 +98,6 @@ LetScoping.prototype.run = function () {
   block._letDone = true;
 
   var needsClosure = this.getLetReferences();
-  this.checkTDZ();
 
   // this is a block within a `Function/Program` so we can safely leave it be
   if (t.isFunction(this.parent) || t.isProgram(this.block)) return;
@@ -113,48 +112,6 @@ LetScoping.prototype.run = function () {
   }
 };
 
-/**
- * Description
- */
-
-LetScoping.prototype.checkTDZ = function () {
-  var state = {
-    letRefs: this.letReferences,
-    file: this.file
-  };
-
-  traverse(this.block, {
-    enter: function (node, parent, scope, context, state) {
-      if (!t.isIdentifier(node)) return;
-      if (!t.isReferenced(node, parent)) return;
-
-      var declared = state.letRefs[node.name];
-      if (!declared) return;
-
-      // declared node is different in this scope
-      if (scope.get(node.name, true) !== declared) return;
-
-      var declaredLoc  = declared.loc;
-      var referenceLoc = node.loc;
-
-      if (!declaredLoc || !referenceLoc) return;
-
-      // does this reference appear on a line before the declaration?
-      var before = referenceLoc.start.line < declaredLoc.start.line;
-
-      if (referenceLoc.start.line === declaredLoc.start.line) {
-        // this reference appears on the same line
-        // check it appears before the declaration
-        before = referenceLoc.start.col < declaredLoc.start.col;
-      }
-
-      if (before) {
-        throw state.file.errorWithNode(node, "Temporal dead zone - accessing a variable before it's initialized");
-      }
-    }
-  }, this.scope, state);
-};
-
 /**
  * Description
  */
@@ -179,7 +136,7 @@ LetScoping.prototype.remap = function () {
       var uid = file.generateUidIdentifier(ref.name, scope).name;
       ref.name = uid;
 
-      remaps[key] = {
+      remaps[key] = remaps[uid] = {
         node: ref,
         uid: uid
       };

lib/6to5/transformation/transformers/optional-block-scoping-tdz.js

@@ -0,0 +1,47 @@
+var traverse = require("../../traverse");
+var t        = require("../../types");
+
+exports.optional = true;
+
+exports.Loop =
+exports.Program =
+exports.BlockStatement = function (node, parent, scope, context, file) {
+  var letRefs = node._letReferences;
+  if (!letRefs) return;
+
+  var state = {
+    letRefs: letRefs,
+    file: file
+  };
+
+  traverse(node, {
+    enter: function (node, parent, scope, context, state) {
+      if (!t.isIdentifier(node)) return;
+      if (!t.isReferenced(node, parent)) return;
+
+      var declared = state.letRefs[node.name];
+      if (!declared) return;
+
+      // declared node is different in this scope
+      if (scope.get(node.name, true) !== declared) return;
+
+      var declaredLoc  = declared.loc;
+      var referenceLoc = node.loc;
+
+      if (!declaredLoc || !referenceLoc) return;
+
+      // does this reference appear on a line before the declaration?
+      var before = referenceLoc.start.line < declaredLoc.start.line;
+
+      if (referenceLoc.start.line === declaredLoc.start.line) {
+        // this reference appears on the same line
+        // check it appears before the declaration
+        before = referenceLoc.start.col < declaredLoc.start.col;
+      }
+
+      if (before) {
+        throw state.file.errorWithNode(node, "Temporal dead zone - accessing a variable before it's initialized");
+      }
+    }
+  }, scope, state);
+};

package.json

@@ -1,7 +1,7 @@
 {
   "name": "6to5",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "2.13.1",
+  "version": "2.13.2",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://6to5.org/",
   "repository": "6to5/6to5",

test/fixtures/transformation/es6-let-scoping/temporal-dead-zone/options.json

@@ -1,3 +1,4 @@
 {
-  "throws": "Temporal dead zone - accessing a variable before it's initialized"
+  "throws": "Temporal dead zone - accessing a variable before it's initialized",
+  "optional": ["blockScopingTDZ"]
 }