Add "external" prefix to methods and constructors.

Add "external" prefix to methods and constructors.

This is expected to be added to the language specification, for allowing the specification of methods with externally bound implementations.

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

[floitsch, 2012-07-06 15:12:14]

LGTM.

[ahe, 2012-08-02 20:14:36]

I'd like to get this cleaned up.

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
File lib/compiler/implementation/scanner/listener.dart (left):

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
lib/compiler/implementation/scanner/listener.dart:1417: handleModifiers(1);
This is a hack.

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
File lib/compiler/implementation/scanner/listener.dart (right):

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
lib/compiler/implementation/scanner/listener.dart:1417: if
(startKeyword.stringValue == 'external') {
The hack no longer works.

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
File lib/compiler/implementation/scanner/parser.dart (left):

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
lib/compiler/implementation/scanner/parser.dart:570: if (optional('factory',
token)) {
This is kinda of a hack.

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
File lib/compiler/implementation/scanner/parser.dart (right):

https://chromiumcodereview.appspot.com/10697096/diff/1/lib/compiler/implement...
lib/compiler/implementation/scanner/parser.dart:574: (value === 'external' &&
optional('factory', token.next))) {
And now the hack doesn't work anymore.

[ahe, 2012-08-23 16:06:05]

I'll clean this up myself.