Re: [9] RFR of 8129499: Structure of java/rmi/activation/rmidViaInheritedChannel tests masks exception

2015-06-22 Thread Chris Hegarty
+1 -Chris > On 22 Jun 2015, at 21:10, Roger Riggs wrote: > > Hi Brian, > > Looks ok. > > Roger > > >> On 6/22/2015 4:08 PM, Brian Burkhalter wrote: >> Please review at your convenience. >> >> Issue:https://bugs.openjdk.java.net/browse/JDK-8129499 >> Patch:http://cr.openjdk.java.net

Re: [9] RFR of 8129499: Structure of java/rmi/activation/rmidViaInheritedChannel tests masks exception

2015-06-22 Thread Roger Riggs
Hi Brian, Looks ok. Roger On 6/22/2015 4:08 PM, Brian Burkhalter wrote: Please review at your convenience. Issue: https://bugs.openjdk.java.net/browse/JDK-8129499 Patch: http://cr.openjdk.java.net/~bpb/8129499/webrev.00/ Summary: The instance variable ‘rmid’ is never initialized due to so

[9] RFR of 8129499: Structure of java/rmi/activation/rmidViaInheritedChannel tests masks exception

2015-06-22 Thread Brian Burkhalter
Please review at your convenience. Issue: https://bugs.openjdk.java.net/browse/JDK-8129499 Patch: http://cr.openjdk.java.net/~bpb/8129499/webrev.00/ Summary: The instance variable ‘rmid’ is never initialized due to some error which occurs before the statement which would initialize it but it i