On Tue, 29 Oct 2024 15:18:42 GMT, Alexey Ivanov <[email protected]> wrote:

> After updating documentation for `PassFailJFrame.java` in 
> [JDK-8340785](https://bugs.openjdk.org/browse/JDK-8340785) / #21162, jtreg 
> interprets the file as a test because it finds `@test` tag in the file. This 
> results in a failed test.
> 
> I didn't find a way to avoid this problem when using [`@snippet` 
> feature](https://docs.oracle.com/en/java/javase/21/javadoc/programmers-guide-snippets.html#GUID-1B435AA6-84D5-4693-AF5E-6470EEE970FC)
>  of the javadoc (see also [JavaDoc Code Snippet API - Sip of 
> Java](https://inside.java/2022/04/04/sip46/), so I used the old friends 
> `<pre><code>`.
> 
> Now, `@` is represented as an HTML character entity, and jtreg no longer 
> finds the `@test` marker in the file.
> 
> With this update, I inlined the end comment into the same code block with `/` 
> represented as a character entity. This makes the jtreg tags comment 
> complete, ready to copy and paste.
> 
> I've updated the generated javadoc for 
> [`PassFailJFrame`](https://cr.openjdk.org/~aivanov/PassFailJFrame/api/PassFailJFrame.html);
>  this version includes javadoc for 
> [`WindowLayouts`](https://cr.openjdk.org/~aivanov/PassFailJFrame/api/WindowLayouts.html).

LGTM

-------------

Marked as reviewed by honkar (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21766#pullrequestreview-2405682778

Reply via email to