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

Jonathan Gibbons jonathan.gibbons at
Mon Jun 24 10:53:48 PDT 2013

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:
> The bug report is here:
> 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

More information about the compiler-dev mailing list