On Fri, 4 Apr 2025 16:51:24 GMT, Nizar Benalla <nbena...@openjdk.org> 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