Updated webrev looks good!

-Sundar

On 14/11/18, 8:25 PM, Hannes Wallnöfer wrote:
Thanks, Sundar.

I uploaded a new webrev with a comment:

http://cr.openjdk.java.net/~hannesw/8200432/webrev.01/

Hannes

Am 13.11.2018 um 16:32 schrieb Sundararajan 
Athijegannathan<[email protected]>:

Looks good.

Minor nit: There could be a source comment for this pattern in Checker.java
+    private final static Pattern arrayPattern = 
Pattern.compile("^[^\\(]+\\[]");

-Sundar

On 13/11/18, 6:52 PM, Hannes Wallnöfer wrote:
Please review:

Issue:
https://bugs.openjdk.java.net/browse/JDK-8200432

Webrev:
http://cr.openjdk.java.net/~hannesw/8200432/webrev.00/


Thanks,
Hannes

Reply via email to