in acorn_loose/statement.js recovered node for bogus import statements could be improved

[othomann]

I am using acorn 3.1.0, but this is still in master.
The following code reports an error when used with infer.js (from Tern) because there is no local for the corresponding import statement specifier.

   import * from "./exports"

I propose to replace line 378:

 if (this.eatContextual("as")) elt.local = this.parseIdent()

with:

elt.local = this.eatContextual("as") ? this.parseIdent() : this.dummyIdent();

Doing this we make sure that the specifier has a local (recovered in this case). acorn will properly reports a syntax error on the 'from' and infer.js won't fail anymore.

infer.js has the following code:

    ImportDeclaration: function(node, scope) {
      for (var i = 0; i < node.specifiers.length; i++)
        addVar(scope, node.specifiers[i].local)
    }

So without a local being set to a recovered node, addVar fails with: TypeError: Cannot read property 'name' of undefined.

[RReverser]

Sounds good to me. But really, it's better to just send a PR instead of an issue when you already identified what needs to be changed. Would you want to do that?

[othomann]

Ok, I'll prepare one. Thanks.