Hello, Yuri. The updated version look good to me.
With best regards. Petr. On 29 мая 2014 г., at 18:03, Yuri Nesterenko <[email protected]> wrote: > Oops! Thank you Petr, > > here's an updated version with RuntimeException instead of exit(): > > http://cr.openjdk.java.net/~yan/8044172/webrev.01 > > Link to the bug, for consistency, > https://bugs.openjdk.java.net/browse/JDK-8044172 > > Regards, > -yan > > On 05/28/2014 10:18 PM, Petr Pchelko wrote: >> Hello, Yuri. >> >> The bug4523758 test contains System.exit. Could you please replace it with >> throwing an exception? >> Otherwise the fix looks good. >> >> With best regards. Petr. >> >> On May 28, 2014, at 7:34 PM, Yuri Nesterenko <[email protected]> >> wrote: >> >>> Hi, >>> >>> yet another small batch of 2 tests to move from functional suite to jdk >>> >>> Bug is >>> https://bugs.openjdk.java.net/browse/JDK-8044172 >>> >>> and webrev >>> http://cr.openjdk.java.net/~yan/8044172/webrev.00 >>> >>> Thanks! >>> -yan >>> >> >
