Browse Source

Include all declared variables.

The comment suggest multi-variable declarations are split up, but that doesn’t
appear to be the case.
better-aggressive
Mike Bostock 9 years ago
parent
commit
f2333fd42b
  1. 5
      src/ast/attachScopes.js

5
src/ast/attachScopes.js

@ -17,11 +17,10 @@ export default function attachScopes ( statement ) {
scope.addDeclaration( node, false, false ); scope.addDeclaration( node, false, false );
} }
// var foo = 1 // var foo = 1, bar = 2
if ( node.type === 'VariableDeclaration' ) { if ( node.type === 'VariableDeclaration' ) {
const isBlockDeclaration = blockDeclarations[ node.kind ]; const isBlockDeclaration = blockDeclarations[ node.kind ];
// only one declarator per block, because we split them up already node.declarations.forEach( declaration => scope.addDeclaration( declaration, isBlockDeclaration, true ) );
scope.addDeclaration( node.declarations[0], isBlockDeclaration, true );
} }
let newScope; let newScope;

Loading…
Cancel
Save