RFR(s): 8217405 rmic should reject class files with preview features enabled

Lance Andersen lance.andersen at oracle.com
Thu Apr 11 02:00:39 UTC 2019


Looks OK Stuart :-)
> On Apr 10, 2019, at 9:22 PM, Stuart Marks <stuart.marks at oracle.com> wrote:
> 
> Hi all,
> 
> Please review this small fix to rmic to make it reject class files that were compiled with preview features enabled.
> 
> Bug:
> 
> 	https://bugs.openjdk.java.net/browse/JDK-8217405
> 
> Webrev:
> 
> 	http://cr.openjdk.java.net/~smarks/reviews/8217405/webrev.0/
> 
> Note that there is no regression test with this patch; I've added the noreg-hard label. The main reason is that to test this properly requires multiple different JDK versions to be available simultaneously, which isn't supported in out the test environment. One could work around this, e.g. by constructing class files with the desired version information, from but this hardly seems worth it for something like rmic.
> 
> Thanks,
> 
> s'marks

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list