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>

Reply via email to