Re: Code Review Request 7142596: RMI JPRT tests are failing

2012-07-09 Thread Darryl Mocek
Stuart, see inline... On Fri 06 Jul 2012 11:47:54 AM PDT, Stuart Marks wrote: Hi, I've reviewed the first half of the files, thru test/java/rmi/registry/reexport/Reexport.java. Basically things look good and make sense, but there are some details to be ironed out and some cleanups to be do

hg: jdk8/tl/jdk: 7179305: (fs) Method name sun.nio.fs.UnixPath.getPathForExecptionMessage is misspelled

2012-07-09 Thread rob . mckenna
Changeset: 516e0c884af2 Author:robm Date: 2012-07-09 22:26 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/516e0c884af2 7179305: (fs) Method name sun.nio.fs.UnixPath.getPathForExecptionMessage is misspelled Reviewed-by: dholmes, chegar ! src/solaris/classes/sun/nio/fs/LinuxU

Re: Code Review Request 7142596: RMI JPRT tests are failing

2012-07-09 Thread Stuart Marks
OK, here's the review for the second half of the files in the webrev. I saw your reply to the first half (to which I'll reply separately), and I don't think there's anything here that's affected by them. *** AppleUserImpl.java *** ApplicationServer.java REGISTRY_PORT should be a local variab

Re: Code Review Request 7142596: RMI JPRT tests are failing

2012-07-09 Thread Stuart Marks
On 7/9/12 11:14 AM, Darryl Mocek wrote: *** SetChildEnv.java The testing of the message string from the IOException causes me great concern. This message is defined all the way over in java.io.PipedInputStream, and while it's not localized, it does seem like a pretty fragile dependency. I mean,

Re: Code Review Request 7142596: RMI JPRT tests are failing

2012-07-09 Thread Darryl Mocek
On Mon 09 Jul 2012 05:02:23 PM PDT, Stuart Marks wrote: On 7/9/12 11:14 AM, Darryl Mocek wrote: *** SetChildEnv.java The testing of the message string from the IOException causes me great concern. This message is defined all the way over in java.io.PipedInputStream, and while it's not loc

[7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-09 Thread Joe Wang
Hi Paul, I'm back from vacation. You're right. But such an error is also expected. The original design never tried to out-do the java.net.URL. If a system ID input fails URL, it shall result in an exception. The patch that supplied the extra encoding was provided to both Sun and Apache, a

Re: Code Review Request 7142596: RMI JPRT tests are failing

2012-07-09 Thread Stuart Marks
On 7/9/12 5:04 PM, Darryl Mocek wrote: I originally had getUnusedRandomPort/createRegistry(randomPort), but Alan felt LocateRegistry.createRegistry(0) is a better choice. getUnusedRandomPort creates a socket to ensure the port is available, then closes it and returns the port number. It's possibl

Re: [7u8] [Re: RFR [7u6]: 7166896: DocumentBuilder.parse(String uri) is not IPv6 enabled. It throws MalformedURLException

2012-07-09 Thread Paul Sandoz
Hi Joe, What happens when someone logs a bug for system IDs containing IPv6 addresses and non-percent encoded international characters? On Jul 10, 2012, at 3:42 AM, Joe Wang wrote: > Hi Paul, > > I'm back from vacation. > > You're right. But such an error is also expected. The original desig

Re: Code Review Request (7161503 subcase) 7142596: RMI JPRT tests are failing

2012-07-09 Thread Olivier Lagneau
Le 10/07/2012 08:49, Olivier Lagneau a écrit : Hi Stuart, I am in vacation for 2 days, but I think I need reply anyway now. First thanks for reviewing and giving your time on this. Now in the 7161503 SetChilEnv case: *** SetChildEnv.java The testing of the message string from the IOExcepti