Get back recording 'implements' and 'extends' keyword on NodeList

Get back recording 'implements' and 'extends' keyword on NodeList
as suggested by Peter in https://chromiumcodereview.appspot.com/10786028/

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

[ahe, 2012-08-03 06:46:43]

LGTM!

https://chromiumcodereview.appspot.com/10832129/diff/2001/lib/compiler/implem...
File lib/compiler/implementation/tree/unparser.dart (right):

https://chromiumcodereview.appspot.com/10832129/diff/2001/lib/compiler/implem...
lib/compiler/implementation/tree/unparser.dart:129: add(node.token.value);
Long term, consider if this is simpler using addToken here. With the current
implementation, that would sometimes add extra spaces, but you could change
addToken to something like:

bool previousTokenRequiresSpace = false;
void addToken(Token token) {
  if (token === null) return;
  bool requiresSpace = token.kind === KEYWORD_TOKEN || token.kind ===
IDENTIFIER_TOKEN;
  if (previousTokenRequiresSpace && requiresSpace) {
    sb.add(' ');
  }
  previousTokenRequiresSpace = requiresSpace;
  add(token.value);
}

https://chromiumcodereview.appspot.com/10832129/diff/2001/lib/compiler/implem...
lib/compiler/implementation/tree/unparser.dart:176: // TODO(ahe): handle
'const'.
Is this TODO obsolete?

[Roman, 2012-08-03 08:02:19]

https://chromiumcodereview.appspot.com/10832129/diff/2001/lib/compiler/implem...
File lib/compiler/implementation/tree/unparser.dart (right):

https://chromiumcodereview.appspot.com/10832129/diff/2001/lib/compiler/implem...
lib/compiler/implementation/tree/unparser.dart:176: // TODO(ahe): handle
'const'.
On 2012/08/03 06:46:43, ahe wrote:
> Is this TODO obsolete?

I guess so. Removed.