On 8/29/17 11:42 AM, Jonathan Gibbons wrote:
Please review these mostly simple changes to address some
accessibility and other minor HTML issues in the java.corba module.
JBS: https://bugs.openjdk.java.net/browse/JDK-8186924
Webrev: http://cr.openjdk.java.net/~jjg/8186924/webrev.00/
API:
http://cr.openjdk.java.net/~jjg/8186924/api.00/java.corba-summary.html
-- Jon
Notes on changes:
src/java.corba/share/classes/org/omg/CORBA/ORB.java
Update table for accessibility
Not related to your change. The colon in the caption looks strange and
perhaps should be taken out. Well, java.corba is deprecated for removal
and might just leave it as is.
89 * <CAPTION>Standard Java CORBA Properties:</CAPTION>
src/java.corba/share/classes/org/omg/CORBA/TCKind.java
Fix broken HTML, by enclosing '.' in {@code} instead of <code>.
Good catch. This is hard to find through review.
src/java.corba/share/classes/org/omg/CORBA/doc-files/compliance.html
Fix HTML5 errors, and set default language
src/java.corba/share/classes/org/omg/CORBA/doc-files/generatedfiles.html
Fix HTML5 errors, and set default language
src/java.corba/share/classes/org/omg/PortableInterceptor/Interceptors.idl
Fix two tables for accessibility, and remove duplicated text which
was probably
the result of a bad merge in times long ago. In the API, the
modified classes
are CloientRequestInfo and ServerRequestInfo.
Otherwise, look fine to me.
Mandy
