Properly emit variable declarations.

Properly emit variable declarations.

VariableElements share definition in VarialbleListElement, so it should be
only output once.

R=smok@google.com

Committed: https://code.google.com/p/dart/source/detail?r=10082

[Roman, 2012-07-31 18:45:03]

lgtm

https://chromiumcodereview.appspot.com/10837025/diff/1/lib/compiler/implement...
File lib/compiler/implementation/dart_backend/emitter.dart (right):

https://chromiumcodereview.appspot.com/10837025/diff/1/lib/compiler/implement...
lib/compiler/implementation/dart_backend/emitter.dart:74: if
(!processedVariableLists.contains(variableList)) {
Hmm, I just thought about it in another way - we output the whole list of
defined variables, even the ones we don't need, right?
Maybe a TODO to fix it? (you may assign it to me if you want) Although the fix
will probably require some duplication of unparser code

[Anton Muhin, 2012-07-31 18:55:37]

https://chromiumcodereview.appspot.com/10837025/diff/1/lib/compiler/implement...
File lib/compiler/implementation/dart_backend/emitter.dart (right):

https://chromiumcodereview.appspot.com/10837025/diff/1/lib/compiler/implement...
lib/compiler/implementation/dart_backend/emitter.dart:74: if
(!processedVariableLists.contains(variableList)) {
Good point, todo added.

On 2012/07/31 18:45:03, Roman wrote:
> Hmm, I just thought about it in another way - we output the whole list of
> defined variables, even the ones we don't need, right?
> Maybe a TODO to fix it? (you may assign it to me if you want) Although the fix
> will probably require some duplication of unparser code