cerxes/babel
3
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #2866 from hzoo/i-2865

Browse Source 
transform-es2015-typeof-symbol: check for undefined - fixes #2865
This commit is contained in:
Sebastian McKenzie
2015-11-06 19:40:21 -05:00
parent d7adedad3c bb5a7804bc
commit a2202802e4
4 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
packages/babel-core/test/fixtures/transformation/transform-es2015-typeof-symbol/non-typeof/actual.js vendored Normal file
View File

@@ -0,0 +1,5 @@
export default function (number) {
if (!isNaN(number)) {
return 1;
}
}

5
packages/babel-core/test/fixtures/transformation/transform-es2015-typeof-symbol/non-typeof/expected.js vendored Normal file
View File

@@ -0,0 +1,5 @@
export default function (number) {
if (!isNaN(number)) {
return 1;
}
}

3
packages/babel-core/test/fixtures/transformation/transform-es2015-typeof-symbol/options.json vendored Normal file
View File

@@ -0,0 +1,3 @@
{
"plugins": ["transform-es2015-typeof-symbol"]
}

2
packages/babel-plugin-transform-es2015-typeof-symbol/src/index.js
View File

@@ -6,7 +6,7 @@ export default function ({ types: t }) {
UnaryExpression(path) {
let { node, parent } = path;
if (node[IGNORE]) return;
if (path.find(path => !!path.node._generated)) return;
if (path.find(path => path.node && !!path.node._generated)) return;
if (path.parentPath.isBinaryExpression() && t.EQUALITY_BINARY_OPERATORS.indexOf(parent.operator) >= 0) {
// optimise `typeof foo === "string"` since we can determine that they'll never need to handle symbols