On 7/11/13 3:28 PM, Seán Coffey wrote: > 8019979 : Replace CheckPackageAccess test with better one from closed repo > > I'd like to push this testcase enhancement to 7u40. It was pushed to jdk8 TL > repo earlier in the week. I've run tests via JPRT and saw no issues. The > package list in testcase is slightly different to that in jdk8 given the > package differences that exist between 7u & 8. > > Sean - can I ask you to review the change again ? A second review would be > welcome also. > > bug report : http://bugs.sun.com/view_bug.do?bug_id=8019979 > jdk8 review : > http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-July/018811.html > 7u40 webrev : http://cr.openjdk.java.net/~coffeys/webrev.8019979.7u40/webrev/
Looks good, approved! cheers, dalibor topic -- Oracle <http://www.oracle.com> Dalibor Topic | Principal Product Manager Phone: +494089091214 <tel:+494089091214> | Mobile: +491737185961 <tel:+491737185961> Oracle Java Platform Group ORACLE Deutschland B.V. & Co. KG | Kühnehöfe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 München Registergericht: Amtsgericht München, HRA 95603 Geschäftsführer: Jürgen Kunz Komplementärin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher Green Oracle <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment
