RFR(XS): 8150077: Due to a javac type inference issue, javadoc doesn't compile with a jdk prior to 8u40

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR(XS): 8150077: Due to a javac type inference issue, javadoc doesn't compile with a jdk prior to 8u40

Volker Simonis
Hi,

can I please have a review for the following small change which fixes
two javac compilation errors during the build of javadoc if the boot
jdk is older than 8u40:

http://cr.openjdk.java.net/~simonis/webrevs/2016/8150077/
https://bugs.openjdk.java.net/browse/JDK-8150077

The first compilation error is copied into the bug report.

I've actually only tested with 8u20 (which doesn't work) and 8u74
(which does work) but the javac errors are similar to the ones in
"8087115: Due to a javac type inference issue, sjavac doesn't compile
with 8u31" (https://bugs.openjdk.java.net/browse/JDK-8087115) which
mentions 8u40 as the first version which correctly handles such code.

Thank you and best regards,
Volker
Reply | Threaded
Open this post in threaded view
|

Re: RFR(XS): 8150077: Due to a javac type inference issue, javadoc doesn't compile with a jdk prior to 8u40

Jonathan Gibbons
Looks OK to me.

-- Jon

On 02/17/2016 10:26 AM, Volker Simonis wrote:

> Hi,
>
> can I please have a review for the following small change which fixes
> two javac compilation errors during the build of javadoc if the boot
> jdk is older than 8u40:
>
> http://cr.openjdk.java.net/~simonis/webrevs/2016/8150077/
> https://bugs.openjdk.java.net/browse/JDK-8150077
>
> The first compilation error is copied into the bug report.
>
> I've actually only tested with 8u20 (which doesn't work) and 8u74
> (which does work) but the javac errors are similar to the ones in
> "8087115: Due to a javac type inference issue, sjavac doesn't compile
> with 8u31" (https://bugs.openjdk.java.net/browse/JDK-8087115) which
> mentions 8u40 as the first version which correctly handles such code.
>
> Thank you and best regards,
> Volker