On Fri, 4 Apr 2025 16:51:24 GMT, Nizar Benalla <[email protected]> wrote:
>> Hannes Wallnöfer has updated the pull request with a new target base due to
>> a merge or a rebase. The pull request now contains four commits:
>>
>> - Merge branch 'master' into JDK-8350920
>> - Merge branch 'master' into JDK-8350920
>> - Add svg files
>> - 8350920: Allow inherited member summaries to be viewed inline
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/html/RawHtml.java line 162:
>
>> 160: @Override
>> 161: public Content stripTags() {
>> 162: return Text.of(rawHtmlContent.replaceAll("<[^>]*>", ""));
>
> nit: we use this same transformation `"<[^>]*>"` -> `""` in other places
> (IndexTaglet and SpecTaglet), could we replace those ad-hoc uses with this
> method?
Thanks @nizarbenalla, I replaced the two local implementations of this
functionality in `IndexTaglet` and `SpecTaglet`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23967#discussion_r2032835110
