RFR: JDK-8049522 - Move @implNote in org.omg.CORBA.ORB to init method

Alan Bateman Alan.Bateman at oracle.com
Fri Oct 24 08:04:28 UTC 2014


On 24/10/2014 00:05, Mark Sheppard wrote:
> Hi,
>    please oblige and review the change
> http://cr.openjdk.java.net/~msheppar/8049522/webrev/
>
The note in the ORB.init methods looks good. I think it will read a bit 
more smoother if you drop the comma after "When configured".

For the note in the class description "When a singleton ORB .." should 
probably be "When the singleton ORB ...".

I wonder if this statement could be re-worded or even dropped:

"Thus, where appropriate, it is necessary that an application's 
CLASSPATH includes the classes for this alternative ORBSingleton. It 
should be noted the singleton ORB is system wide."

It hints of the CLASSPATH env variable and also not clear what this 
means for applications running in a container. How about:

"An application started on the command line with the java launcher will 
put the classes for the alternative ORB implementation on the class path 
for example".

-Alan.




More information about the core-libs-dev mailing list