Fix identifier handling on export { default as ... } (#1257)

Author: dcodeIO

src/parser.ts

@@ -2473,7 +2473,7 @@ export class Parser extends DiagnosticEmitter {
 
     // before: Identifier ('as' Identifier)?
 
-    if (tn.skipIdentifier()) {
+    if (tn.skipIdentifier(IdentifierHandling.ALWAYS)) {
       let identifier = Node.createIdentifierExpression(tn.readIdentifier(), tn.range());
       let asIdentifier: IdentifierExpression | null = null;
       if (tn.skip(Token.AS)) {

tests/compiler/reexport.optimized.wat

@@ -25,6 +25,8 @@
  (export "exportstar.renamed_c" (global $export/c))
  (export "exportstar.ns.two" (func $export/ns.one))
  (export "exportstar.default.two" (func $export/ns.one))
+ (export "default" (func $export/ns.one))
+ (export "renamed_default" (func $export/ns.one))
  (func $export/add (param $0 i32) (param $1 i32) (result i32)
   local.get $0
   local.get $1

tests/compiler/reexport.ts

@@ -27,3 +27,6 @@ export { ns as renamed_ns } from "./export";
 
 import * as exportstar from "./exportstar";
 export { exportstar };
+
+export { default } from "./export-default";
+export { default as renamed_default } from "./export-default";

tests/compiler/reexport.untouched.wat

@@ -26,6 +26,8 @@
  (export "exportstar.renamed_c" (global $export/c))
  (export "exportstar.ns.two" (func $export/ns.two))
  (export "exportstar.default.two" (func $export/ns.two))
+ (export "default" (func $export-default/theDefault))
+ (export "renamed_default" (func $export-default/theDefault))
  (start $~start)
  (func $export/add (param $0 i32) (param $1 i32) (result i32)
   local.get $0
@@ -58,6 +60,9 @@
  (func $export/ns.two
   nop
  )
+ (func $export-default/theDefault
+  nop
+ )
  (func $~start
   call $start:reexport
  )

tests/compiler/rereexport.optimized.wat

@@ -1,4 +1,5 @@
 (module
+ (type $none_=>_none (func))
  (memory $0 0)
  (global $export/a i32 (i32.const 1))
  (global $export/b i32 (i32.const 2))
@@ -7,4 +8,9 @@
  (export "renamed_a" (global $export/a))
  (export "renamed_b" (global $export/b))
  (export "renamed_renamed_b" (global $export/b))
+ (export "default" (func $export-default/theDefault))
+ (export "renamed_default" (func $export-default/theDefault))
+ (func $export-default/theDefault
+  nop
+ )
 )

tests/compiler/rereexport.ts

@@ -2,5 +2,7 @@ export {
   a,
   a as renamed_a,
   renamed_b,
-  renamed_b as renamed_renamed_b
+  renamed_b as renamed_renamed_b,
+  default,
+  default as renamed_default
 } from "./reexport";

tests/compiler/rereexport.untouched.wat

@@ -11,6 +11,8 @@
  (export "renamed_a" (global $export/a))
  (export "renamed_b" (global $export/b))
  (export "renamed_renamed_b" (global $export/b))
+ (export "default" (func $export-default/theDefault))
+ (export "renamed_default" (func $export-default/theDefault))
  (start $~start)
  (func $export/add (param $0 i32) (param $1 i32) (result i32)
   local.get $0
@@ -35,6 +37,9 @@
  (func $start:rereexport
   call $start:reexport
  )
+ (func $export-default/theDefault
+  nop
+ )
  (func $~start
   call $start:rereexport
  )

tests/compiler/wildcard-export.optimized.wat

@@ -1,4 +1,5 @@
 (module
+ (type $none_=>_none (func))
  (memory $0 0)
  (global $export/a i32 (i32.const 1))
  (global $export/b i32 (i32.const 2))
@@ -7,4 +8,9 @@
  (export "renamed_a" (global $export/a))
  (export "renamed_b" (global $export/b))
  (export "renamed_renamed_b" (global $export/b))
+ (export "default" (func $export-default/theDefault))
+ (export "renamed_default" (func $export-default/theDefault))
+ (func $export-default/theDefault
+  nop
+ )
 )

tests/compiler/wildcard-export.untouched.wat

@@ -11,6 +11,8 @@
  (export "renamed_a" (global $export/a))
  (export "renamed_b" (global $export/b))
  (export "renamed_renamed_b" (global $export/b))
+ (export "default" (func $export-default/theDefault))
+ (export "renamed_default" (func $export-default/theDefault))
  (start $~start)
  (func $export/add (param $0 i32) (param $1 i32) (result i32)
   local.get $0
@@ -38,6 +40,9 @@
  (func $start:wildcard-export
   call $start:rereexport
  )
+ (func $export-default/theDefault
+  nop
+ )
  (func $~start
   call $start:wildcard-export
  )