Create implicit super constructor call if necessary

Create implicit super constructor call if necessary

Create an implicit call to the super constructor if no explicit
call is present, and also if the constructor does not have an
initializer list at all.

The evaluation order is still not conforming to the spec. Thus,
a couple of tests that happened to pass before now fail.

Fix issue 85
https://code.google.com/p/dart/issues/detail?id=85
Committed: https://code.google.com/p/dart/source/detail?r=412

[regis, 2011-10-13 19:32:29]

LGTM

http://codereview.chromium.org/8286003/diff/2003/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

http://codereview.chromium.org/8286003/diff/2003/runtime/vm/parser.cc#newcode...
runtime/vm/parser.cc:1381: for (;;) {
If you do not consume the colon above (and increment the look ahead count), you
can change this for loop to a more readable form:
do {
  ConsumeToken();
  ...
} while (CurrentToken() == Token::kCOMMA);

But it's fine if you leave it as is.

http://codereview.chromium.org/8286003/diff/2003/runtime/vm/parser.cc#newcode...
runtime/vm/parser.cc:1672: for (;;) {
Same comment here about a do loop instead of a for(;;).
You could delay the consume on line 1818 and start here with
ASSERT(CurrentToken() == Token::kCOLON);

[hausner, 2011-10-13 20:24:09]

Thanks!

http://codereview.chromium.org/8286003/diff/2003/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

http://codereview.chromium.org/8286003/diff/2003/runtime/vm/parser.cc#newcode...
runtime/vm/parser.cc:1381: for (;;) {
Nice idea. Done.

http://codereview.chromium.org/8286003/diff/2003/runtime/vm/parser.cc#newcode...
runtime/vm/parser.cc:1672: for (;;) {
And done.