Better PrettyPrinter output for Return nodes, display tokens and expression.

Better PrettyPrinter output for Return nodes, display tokens and expression.

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

[Roman, 2012-06-14 21:34:26]

You may submit this yourself if I can't do it before vacations.

[Anton Muhin, 2012-06-15 02:55:56]

lgtm w/ comments

https://chromiumcodereview.appspot.com/10546174/diff/4002/lib/compiler/implem...
File lib/compiler/implementation/tree/prettyprint.dart (right):

https://chromiumcodereview.appspot.com/10546174/diff/4002/lib/compiler/implem...
lib/compiler/implementation/tree/prettyprint.dart:245: node.beginToken != null ?
node.beginToken.stringValue : "null";
nit (here and below): !== null

https://chromiumcodereview.appspot.com/10546174/diff/4002/lib/compiler/implem...
lib/compiler/implementation/tree/prettyprint.dart:248: if (node.hasExpression) {
nit: single line if?

[Roman, 2012-06-17 00:27:03]

https://chromiumcodereview.appspot.com/10546174/diff/4002/lib/compiler/implem...
File lib/compiler/implementation/tree/prettyprint.dart (right):

https://chromiumcodereview.appspot.com/10546174/diff/4002/lib/compiler/implem...
lib/compiler/implementation/tree/prettyprint.dart:245: node.beginToken != null ?
node.beginToken.stringValue : "null";
On 2012/06/15 02:55:56, antonmuhin wrote:
> nit (here and below): !== null

Done.

https://chromiumcodereview.appspot.com/10546174/diff/4002/lib/compiler/implem...
lib/compiler/implementation/tree/prettyprint.dart:248: if (node.hasExpression) {
On 2012/06/15 02:55:56, antonmuhin wrote:
> nit: single line if?

Done.

[Anton Muhin, 2012-06-18 10:13:47]

I've landed it on your behalf.  Thanks a lot for a neat tool.