RFR: [15,docs] JDK-8247899,HTML errors and warnings in threadPrimitiveDeprecation.html
David Holmes
david.holmes at oracle.com
Fri Jun 19 06:58:36 UTC 2020
Looks good and trivial!
Thanks,
David
On 19/06/2020 10:18 am, Jonathan Gibbons wrote:
> Please review this trivial fix to some minor issues reported by doclint.
>
> In the first change, the `<hr>` contained a hangover of some HTML 4
> attributes
> which are not supported in HTML5. They are simply deleted.
>
> In the second change, a paragraph just contained a comment. While it might
> be reasonable to just remove the paragraph tags surrounding the comment,
> the comment itself ("Body text ends here") is somewhat redundant, appearing
> as it does right before `</body>` and so the entire line is deleted.
>
> -- Jon
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8247899
>
> Inline patch:
>
> diff -r f80508232687
> src/java.base/share/classes/java/lang/doc-files/threadPrimitiveDeprecation.html
>
> ---
> a/src/java.base/share/classes/java/lang/doc-files/threadPrimitiveDeprecation.html
> Thu Jun 18 16:21:34 2020 -0700
> +++
> b/src/java.base/share/classes/java/lang/doc-files/threadPrimitiveDeprecation.html
> Thu Jun 18 17:10:24 2020 -0700
> @@ -282,7 +282,7 @@
> }
> }
> </pre>
> -<hr size="3" noshade="noshade" />
> +<hr>
> <h2>Can I combine the two techniques to produce a thread that may
> be safely "stopped" or "suspended"?</h2>
> Yes, it's reasonably straightforward. The one subtlety is that the
> @@ -324,6 +324,5 @@
> described above, it needn't call <code>notify</code> as well, but it
> still must be synchronized. This ensures that the target thread
> won't miss an interrupt due to a race condition.
> -<p><!-- Body text ends here --></p>
> </body>
> </html>
>
More information about the core-libs-dev
mailing list