Thanks. I'll tweak the caption.
-- Jon
On 08/31/2017 02:45 PM, mandy chung wrote:
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
