Re: RFR(S) 8222935: Add forgotten files for ExceptionCheckingJniEnv

[Chris Plummer]

Hi JC, Looks good. thanks, Chris On 4/24/19 4:42 PM, Jean Christophe Beyler wrote: Hi all, Submit repo actually failed on the first version due to some loss of precision assignments. And also there was a slight bug in the exception messaging handling, so this is now fixed in the webrev: This version passes the test and the submit repo: Webrev: http://cr.openjdk.java.net/~jcbeyler/8222935/webrev.01/ Bug: https://bugs.openjdk.java.net/browse/JDK-8222935 Thanks for your help, Jc On Wed, Apr 24, 2019 at 11:10 AM Alex Menkov <alexey.men...@oracle.com> wrote: LGTM --alex On 04/24/2019 11:07, Jean Christophe Beyler wrote: > Hi all, > > When pushing my fix for JDK-8213501, I did not push the added files > which were the new test for the whole system. > > These were reviewed via the JDK-8213501 > (http://cr.openjdk.java.net/~jcbeyler/8213501/webrev.08/) fix but I'm > just asking for a review again to add this webrev: > > Webrev: http://cr.openjdk.java.net/~jcbeyler/8222935/webrev.00/ > Bug: https://bugs.openjdk.java.net/browse/JDK-8222935 > > My apologies for this... This has been sent to the submit repo in parallel. > > Thanks, > Jc -- Thanks, Jc