Browse Source

break apart top-level var declarations with multiple declarators - fixes #26

contingency-plan
Rich Harris 9 years ago
parent
commit
207cb9dfba
  1. 45
      src/Module.js

45
src/Module.js

@ -28,12 +28,19 @@ export default class Module {
// Try to extract a list of top-level statements/declarations. If
// the parse fails, attach file info and abort
let ast;
try {
const ast = parse( source, {
ast = parse( source, {
ecmaVersion: 6,
sourceType: 'module',
onComment: ( block, text, start, end ) => this.comments.push({ block, text, start, end })
});
} catch ( err ) {
err.code = 'PARSE_ERROR';
err.file = path;
throw err;
}
walk( ast, {
enter: node => {
@ -42,16 +49,38 @@ export default class Module {
}
});
this.statements = ast.body.map( ( node, i ) => {
const magicString = this.magicString.snip( node.start, node.end ).trim();
return new Statement( node, magicString, this, i );
this.statements = [];
ast.body.map( node => {
// special case - top-level var declarations with multiple declarators
// should be split up. Otherwise, we may end up including code we
// don't need, just because an unwanted declarator is included
if ( node.type === 'VariableDeclaration' && node.declarations.length > 1 ) {
node.declarations.forEach( declarator => {
const magicString = this.magicString.snip( declarator.start, declarator.end ).trim();
magicString.prepend( `${node.kind} ` ).append( ';' );
const syntheticNode = {
type: 'VariableDeclaration',
kind: node.kind,
start: node.start,
end: node.end,
declarations: [ declarator ]
};
const statement = new Statement( syntheticNode, magicString, this, this.statements.length );
this.statements.push( statement );
});
} catch ( err ) {
err.code = 'PARSE_ERROR';
err.file = path;
throw err;
}
else {
const magicString = this.magicString.snip( node.start, node.end ).trim();
const statement = new Statement( node, magicString, this, this.statements.length );
this.statements.push( statement );
}
});
this.importDeclarations = this.statements.filter( isImportDeclaration );
this.exportDeclarations = this.statements.filter( isExportDeclaration );

Loading…
Cancel
Save