On Mon, 24 Mar 2025 14:41:43 GMT, Nizar Benalla <[email protected]> wrote:
> Please review this patch to fix an issue in the `tag-search-index.js`
> generation.
>
> The problem was in the `toJSON()` method of `IndexItem`. When adding the
> description, it's using `escapeQuotes(description)` which only escapes
> backslashes and quotes, but doesn't normalize whitespace.
>
> TIA
src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/IndexItem.java
line 645:
> 643: if (!description.isEmpty()) {
> 644: String normalizedDescription =
> description.replaceAll("\\s+", " ");
> 645: builder.append("d",
> escapeQuotes(normalizedDescription));
Should we just handle all whitespaces in `escapeQuotes`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24202#discussion_r2010492415
