Review request for JDK-8012722: Single comma in array initializer should parse

[Eric McCorkle]

Webrev updated with your suggestion.

On 06/24/13 13:53, Jonathan Gibbons wrote:
> On 06/24/2013 10:24 AM, Eric McCorkle wrote:
>> Hello,
>>
>> Please review this simple patch which updates the javac parser to accept
>> array constants in the form of @Foo({,}) in accordance with JLS 9.7.1.
>>
>> The webrev is here:
>> http://cr.openjdk.java.net/~emc/8012722/
>>
>> The bug report is here:
>> http://bugs.sun.com/view_bug.do?bug_id=8012722
>>
>> Thanks,
>> Eric
> 
> OK, but the negative test would be better if you checked it failed for the
> right reason, perhaps by using -XDrawDiagnostics and a .out file, per
> \the prevailing standard.
> 
> -- Jon
-------------- next part --------------
A non-text attachment was scrubbed...
Name: eric_mccorkle.vcf
Type: text/x-vcard
Size: 314 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20130624/078c4c1c/eric_mccorkle.vcf