On Tue, 8 Apr 2025 18:08:57 GMT, Chen Liang <li...@openjdk.org> wrote:

>> The else block generates the closing tag for the note. As you see in the PR 
>> description, the tag syntax resembles HTML open/close tags. This is a 
>> compromise to be able to locate the note within the description (which isn't 
>> possible with block tags), and being able to use rich content in the note 
>> without being required to writing a dedicated tag parser and rendering code. 
>> (I also happen to find this syntax easier to read than a custom inline tag, 
>> which would require additional rules to separate meta data from note 
>> content.)
>
> Such usage should be reflected in `testPreviewNoteTag`, which currently does 
> not use it as open/close tags.

The problem is the Taglet class is part of the build system, and I don't think 
we have a way to test these build classes (I coulndn't find any tests for the 
other build Taglets). I could replicate the Taglet class in the test, but that 
doesn't seem like a good solution either.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23395#discussion_r2033795505

Reply via email to