RFR: JDK-8237214: fix for JDK-8236597 reintroduced wrong subexpression

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Wed Jan 15 19:14:41 UTC 2020

Looks good


On 15/01/2020 19:10, Vicente Romero wrote:
> Hi,
> Please review this patch that fixes [1] at [2]. The fix for 
> JDK-8237214 that I pushed this morning reintroduced a subexpression at 
> JavacParser that has been removed by another patch. Both patches were 
> based on the same code and both were modifying the same line, then I 
> resurrected a condition that was removed by a previous patch.
> Thanks,
> Vicente
> [1] https://bugs.openjdk.java.net/browse/JDK-8237214
> [2] http://cr.openjdk.java.net/~vromero/8237214/webrev.00/

More information about the compiler-dev mailing list