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

[Jonathan Gibbons]

Looks good. Did you mean the comment to wrap like that?

-- Jon

On 06/24/2013 05:47 PM, Eric McCorkle wrote:
> 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