Re: Review request for JDK-8148148: Remove pluggable CodeStore API

2016-03-07 Thread Hannes Wallnoefer
Thanks for the reviews. I just noticed the original webrev didn't remove unused headers and update the copyright date. I've uploaded a new webrev with those changes added. http://cr.openjdk.java.net/~hannesw/8148148/webrev.01/ Hannes Am 2016-03-07 um 10:37 schrieb Attila Szegedi: +1 On M

Re: Review request for JDK-8148148: Remove pluggable CodeStore API

2016-03-07 Thread Michael Haupt
Hi Hannes, lower-case thumbs up. Best, Michael > Am 07.03.2016 um 10:28 schrieb Hannes Wallnoefer > : > > Please review JDK-8148148: Remove pluggable CodeStore API: > > Webrev: http://cr.openjdk.java.net/~hannesw/8148148/webrev/ > Bug: https://bugs.openjdk.java.net/browse/JDK-8148148 > > Th

Re: Review request for JDK-8148148: Remove pluggable CodeStore API

2016-03-07 Thread Attila Szegedi
+1 > On Mar 7, 2016, at 10:28 AM, Hannes Wallnoefer > wrote: > > Please review JDK-8148148: Remove pluggable CodeStore API: > > Webrev: http://cr.openjdk.java.net/~hannesw/8148148/webrev/ > Bug: https://bugs.openjdk.java.net/browse/JDK-8148148 > > Thanks, > Hannes