Re: RFR 8006996: Add a pretty printer that prints script source in nice form

[A. Sundararajan]

Thanks for the review. Updated and uploaded revised webrev for reference 
: http://cr.openjdk.java.net/~sundar/8006996/webrev.01/

Thanks,
-Sundar
On Thursday 19 March 2015 03:06 PM, Hannes Wallnoefer wrote:
+1

I think "intent" should be "indent" for indentation.

This would be a terser representation of operators, although I realize 
it's a matter of taste:

var operators = {
    POSTFIX_INCREMENT : "++",
    POSTFIX_DECREMENT: "--",
    ...
};

operators[Kind.POSTFIX_INCREMENT]    // can use Java enum values as keys


-- Hannes


Am 2015-03-19 um 05:45 schrieb A. Sundararajan:
Please review http://cr.openjdk.java.net/~sundar/8006996/ for 
https://bugs.openjdk.java.net/browse/JDK-8006996

Thanks,
-Sundar