Martin,
';' can be used either way ... it's an optional terminator on the last
item. w3schools typically shows without the ';' when it is not necessary.
-- Jon
On 04/26/2017 03:40 PM, Martin Buchholz wrote:
This looks good.
jsr166 maintainers will merge into jsr166 CVS.
I'm unsure whether the ";" should be used as separator or terminator.
I would have terminated every style with a ";"
On Wed, Apr 26, 2017 at 3:31 PM, Jonathan Gibbons
<[email protected] <mailto:[email protected]>> wrote:
Please review the following conceptually simple fix to change use
of the align and valign attributes in table-related tags to inline
style attributes in the public doc comments for the contents of
java.base module.
This is for compatibility with HTML 5.
The change was done mechanically with the following sed script,
which was applied to the set of files reported by "javadoc -html5"
as containing the use of the align and valign attribute.
s/ align="*left"*/ style="text-align:left"/
s/ align="*center"*/ style="text-align:center"/
s/ ALIGN="*CENTER"*/ style="text-align:center"/
s/ align="*right"*/ style="text-align:right"/
s/ valign="*top"*/ style="vertical-align:top"/
s/ valign="*bottom"*/ style="vertical-align:bottom"/
s/ style="\([^"]*\)" style="\([^"]*\)"/ style="\1; \2"/
The first six lines change the variants of align and valign into
the corresponding style attributes. And, because there were some
instances where a tag had both align and valign, the final line of
the script merges two instances of the style attribute into a
single instance.
This covers files in the following packages:
java.base java/io
java.base java/lang
java.base java/lang/reflect
java.base java/math
java.base java/nio/charset
java.base java/text
java.base java/time/chrono
java.base java/time/format
java.base java/util
java.base java/util/concurrent
java.base java/util/regex
JBS: https://bugs.openjdk.java.net/browse/JDK-8179367
<https://bugs.openjdk.java.net/browse/JDK-8179367>
webrev: http://cr.openjdk.java.net/~jjg/8179367/webrev/
<http://cr.openjdk.java.net/%7Ejjg/8179367/webrev/>
-- Jon
