+1

Am 2015-12-16 um 11:33 schrieb Sundararajan Athijegannathan:
Thanks for the reviews.

Updated webrev: http://cr.openjdk.java.net/~sundar/8145486/webrev.01/

* Fixed documentation for setIndexedPropertiesToExternalArrayData
* Fixed inner/nested class case ($ - > ".")
* Added doc for NativeJavaPackage objects
* Added missing "final" in Console.bind method.

Incremental changes are only in Main.java and Console.java

Please review..

-Sundar

On 12/16/2015 2:48 PM, Hannes Wallnoefer wrote:
Very nice, and works like a charm on Linux.

One nitpick: documentation for NativeObject.setIndexedPropertiesToExternalArrayData should start with "sets" rather than "set" to be consistent with the rest.

Hannes

Am 2015-12-16 um 05:38 schrieb Sundararajan Athijegannathan:
Please review http://cr.openjdk.java.net/~sundar/8145486/ for https://bugs.openjdk.java.net/browse/JDK-8145486

Note that this enhancement only adds jjs changes to show documentation when available and infrastructure to attach documentation string for JS built-in functions, constructors. Actual documentation strings are added only for NativeObject. Adding documentation string for all other built-in functions, constructors will be done in future.

Thanks,
-Sundar



Reply via email to