v2.9.4

This reverts commit b619e843a9.

CHANGELOG.md

@@ -11,6 +11,16 @@
 
 _Note: Gaps between patch versions are faulty/broken releases._
 
+## 2.9.4
+
+ * **Bug Fix**
+  * Support `UpdateExpression`s as `MemberExpression` objects.
+
+## 2.9.3
+
+ * **Bug Fix**
+  * Remove `exportsWildcard` helper in favor of `defaults`.
+
 ## 2.9.2
 
  * **Bug Fix**

Makefile

@@ -71,6 +71,9 @@ publish:
 
 	make test
 
+	read -p "Version: "  version; \
+	npm version $$version --message "v%s"
+
 	make build
 	cp dist/6to5.min.js browser.js
 	cp dist/polyfill.min.js browser-polyfill.js
@@ -79,8 +82,6 @@ publish:
 	node bin/cache-templates
 	test -f templates.json
 
-	read -p "Version: "  version; \
-  npm version $$version --message "v%s"
 	npm publish
 
 	git push --follow-tags

lib/6to5/file.js

@@ -35,7 +35,6 @@ File.helpers = [
   "async-to-generator",
   "interop-require-wildcard",
   "typeof",
-  "exports-wildcard",
   "extends",
   "get"
 ];

lib/6to5/generation/node/parentheses.js

@@ -21,6 +21,13 @@ _.each([
   });
 });
 
+exports.UpdateExpression = function (node, parent) {
+  if (t.isMemberExpression(parent) && parent.object === node) {
+    // (foo++).test()
+    return true;
+  }
+};
+
 exports.ObjectExpression = function (node, parent) {
   if (t.isExpressionStatement(parent)) {
     // ({ foo: "bar" });

lib/6to5/transformation/modules/_default.js

@@ -217,9 +217,9 @@ DefaultFormatter.prototype._exportSpecifier = function (getRef, specifier, node,
 };
 
 DefaultFormatter.prototype._exportsWildcard = function (objectIdentifier) {
-  return t.expressionStatement(t.callExpression(this.file.addHelper("exports-wildcard"), [
-    t.callExpression(this.file.addHelper("interop-require-wildcard"), [objectIdentifier]),
-    t.identifier("exports")
+  return t.expressionStatement(t.callExpression(this.file.addHelper("defaults"), [
+    t.identifier("exports"),
+    t.callExpression(this.file.addHelper("interop-require-wildcard"), [objectIdentifier])
   ]));
 };
 

lib/6to5/transformation/templates/exports-wildcard.js

@@ -1,7 +0,0 @@
-(function (obj, exports) {
-  for (var i in obj) {
-    if (exports[i] !== undefined) {
-      exports[i] = obj[i];
-    }
-  }
-})

lib/6to5/transformation/transformers/es6-destructuring.js

@@ -134,7 +134,7 @@ var pushPattern = function (opts) {
   var file = opts.file;
   var scope = opts.scope;
 
-  if (!t.isMemberExpression(parentId) && !t.isIdentifier(parentId)) {
+  if (!t.isArrayExpression(parentId) && !t.isMemberExpression(parentId) && !t.isIdentifier(parentId)) {
     var key = scope.generateUidBasedOnNode(parentId, file);
 
     nodes.push(t.variableDeclaration("var", [

package.json

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

test/fixtures/generation/types/UpdateExpression/actual.js

@@ -1,2 +1,3 @@
 ++i;
 i++;
+(foo++).test();

test/fixtures/generation/types/UpdateExpression/expected.js

@@ -1,2 +1,3 @@
 ++i;
 i++;
+(foo++).test();

test/fixtures/transformation/es6-constants/destructuring/expected.js

@@ -1,38 +1,18 @@
 "use strict";
 
-var _slicedToArray = function (arr, i) {
-  if (Array.isArray(arr)) {
-    return arr;
-  } else {
-    var _arr = [];
-
-    for (var _iterator = arr[Symbol.iterator](), _step; !(_step = _iterator.next()).done;) {
-      _arr.push(_step.value);
-
-      if (i && _arr.length === i) break;
-    }
-
-    return _arr;
-  }
-};
-
 var _ref = [1, 2];
 
-var _ref2 = _slicedToArray(_ref, 2);
+var a = _ref[0];
+var b = _ref[1];
+var _ref2 = [3, 4];
 
-var a = _ref2[0];
-var b = _ref2[1];
-var _ref3 = [3, 4];
+var c = _ref2[0];
+var d = _ref2[1];
+var _ref3 = { e: 5, f: 6 };
 
-var _ref3 = _slicedToArray(_ref3, 2);
+var e = _ref3.e;
+var f = _ref3.f;
+var _ref4 = { a: 7, b: 8 };
 
-var c = _ref3[0];
-var d = _ref3[1];
-var _ref4 = { e: 5, f: 6 };
-
-var e = _ref4.e;
-var f = _ref4.f;
-var _ref5 = { a: 7, b: 8 };
-
-var g = _ref5.a;
-var h = _ref5.b;
+var g = _ref4.a;
+var h = _ref4.b;

test/fixtures/transformation/es6-destructuring/array/expected.js

@@ -18,13 +18,11 @@ var _slicedToArray = function (arr, i) {
 
 var _ref = ["hello", [", ", "junk"], ["world"]];
 
-var _ref2 = _slicedToArray(_ref, 4);
+var a = _ref[0];
+var _ref$1 = _slicedToArray(_ref[1], 1);
 
-var a = _ref2[0];
-var _ref2$1 = _slicedToArray(_ref2[1], 1);
+var b = _ref$1[0];
+var _ref$2 = _slicedToArray(_ref[2], 1);
 
-var b = _ref2$1[0];
-var _ref2$2 = _slicedToArray(_ref2[2], 1);
-
-var c = _ref2$2[0];
-var d = _ref2[3];
+var c = _ref$2[0];
+var d = _ref[3];

test/fixtures/transformation/es6-destructuring/empty/expected.js

@@ -18,13 +18,11 @@ var _slicedToArray = function (arr, i) {
 
 var _ref = ["foo", "hello", [", ", "junk"], ["world"]];
 
-var _ref2 = _slicedToArray(_ref, 5);
+var a = _ref[1];
+var _ref$2 = _slicedToArray(_ref[2], 1);
 
-var a = _ref2[1];
-var _ref2$2 = _slicedToArray(_ref2[2], 1);
+var b = _ref$2[0];
+var _ref$3 = _slicedToArray(_ref[3], 1);
 
-var b = _ref2$2[0];
-var _ref2$3 = _slicedToArray(_ref2[3], 1);
-
-var c = _ref2$3[0];
-var d = _ref2[4];
+var c = _ref$3[0];
+var d = _ref[4];

test/fixtures/transformation/es6-modules-amd-strict/exports-from/expected.js

@@ -7,15 +7,17 @@ define(["exports", "foo"], function (exports, _foo) {
     };
   };
 
-  var _exportsWildcard = function (obj, exports) {
-    for (var i in obj) {
-      if (exports[i] !== undefined) {
-        exports[i] = obj[i];
+  var _defaults = function (obj, defaults) {
+    for (var key in defaults) {
+      if (obj[key] === undefined) {
+        obj[key] = defaults[key];
       }
     }
+
+    return obj;
   };
 
-  _exportsWildcard(_interopRequireWildcard(_foo), exports);
+  _defaults(exports, _interopRequireWildcard(_foo));
 
   exports.foo = _foo.foo;
   exports.foo = _foo.foo;

test/fixtures/transformation/es6-modules-amd/exports-from/expected.js

@@ -7,15 +7,17 @@ define(["exports", "foo"], function (exports, _foo) {
     };
   };
 
-  var _exportsWildcard = function (obj, exports) {
-    for (var i in obj) {
-      if (exports[i] !== undefined) {
-        exports[i] = obj[i];
+  var _defaults = function (obj, defaults) {
+    for (var key in defaults) {
+      if (obj[key] === undefined) {
+        obj[key] = defaults[key];
       }
     }
+
+    return obj;
   };
 
-  _exportsWildcard(_interopRequireWildcard(_foo), exports);
+  _defaults(exports, _interopRequireWildcard(_foo));
 
   exports.foo = _foo.foo;
   exports.foo = _foo.foo;

test/fixtures/transformation/es6-modules-common-strict/exports-from/expected.js

@@ -6,15 +6,17 @@ var _interopRequireWildcard = function (obj) {
   };
 };
 
-var _exportsWildcard = function (obj, exports) {
-  for (var i in obj) {
-    if (exports[i] !== undefined) {
-      exports[i] = obj[i];
+var _defaults = function (obj, defaults) {
+  for (var key in defaults) {
+    if (obj[key] === undefined) {
+      obj[key] = defaults[key];
     }
   }
+
+  return obj;
 };
 
-_exportsWildcard(_interopRequireWildcard(require("foo")), exports);
+_defaults(exports, _interopRequireWildcard(require("foo")));
 
 exports.foo = require("foo").foo;
 exports.foo = require("foo").foo;

test/fixtures/transformation/es6-modules-common/exports-from/expected.js

@@ -6,15 +6,17 @@ var _interopRequireWildcard = function (obj) {
   };
 };
 
-var _exportsWildcard = function (obj, exports) {
-  for (var i in obj) {
-    if (exports[i] !== undefined) {
-      exports[i] = obj[i];
+var _defaults = function (obj, defaults) {
+  for (var key in defaults) {
+    if (obj[key] === undefined) {
+      obj[key] = defaults[key];
     }
   }
+
+  return obj;
 };
 
-_exportsWildcard(_interopRequireWildcard(require("foo")), exports);
+_defaults(exports, _interopRequireWildcard(require("foo")));
 
 exports.foo = require("foo").foo;
 exports.foo = require("foo").foo;

test/fixtures/transformation/es6-modules-umd-strict/exports-from/expected.js

@@ -13,15 +13,17 @@
     };
   };
 
-  var _exportsWildcard = function (obj, exports) {
-    for (var i in obj) {
-      if (exports[i] !== undefined) {
-        exports[i] = obj[i];
+  var _defaults = function (obj, defaults) {
+    for (var key in defaults) {
+      if (obj[key] === undefined) {
+        obj[key] = defaults[key];
       }
     }
+
+    return obj;
   };
 
-  _exportsWildcard(_interopRequireWildcard(_foo), exports);
+  _defaults(exports, _interopRequireWildcard(_foo));
 
   exports.foo = _foo.foo;
   exports.foo = _foo.foo;

test/fixtures/transformation/es6-modules-umd/exports-from/expected.js

@@ -13,15 +13,17 @@
     };
   };
 
-  var _exportsWildcard = function (obj, exports) {
-    for (var i in obj) {
-      if (exports[i] !== undefined) {
-        exports[i] = obj[i];
+  var _defaults = function (obj, defaults) {
+    for (var key in defaults) {
+      if (obj[key] === undefined) {
+        obj[key] = defaults[key];
       }
     }
+
+    return obj;
   };
 
-  _exportsWildcard(_interopRequireWildcard(_foo), exports);
+  _defaults(exports, _interopRequireWildcard(_foo));
 
   exports.foo = _foo.foo;
   exports.foo = _foo.foo;

test/fixtures/transformation/optional-core-aliasing/es6-destructuring/expected.js

@@ -24,13 +24,11 @@ var _core = _interopRequire(require("core-js/library"));
 
 var _ref = ["hello", [", ", "junk"], ["world"]];
 
-var _ref2 = _slicedToArray(_ref, 4);
+var a = _ref[0];
+var _ref$1 = _slicedToArray(_ref[1], 1);
 
-var a = _ref2[0];
-var _ref2$1 = _slicedToArray(_ref2[1], 1);
+var b = _ref$1[0];
+var _ref$2 = _slicedToArray(_ref[2], 1);
 
-var b = _ref2$1[0];
-var _ref2$2 = _slicedToArray(_ref2[2], 1);
-
-var c = _ref2$2[0];
-var d = _ref2[3];
+var c = _ref$2[0];
+var d = _ref[3];