On Mon, 24 Mar 2025 16:14:54 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:

>> 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`?
>
> I thought about it but that may not be necessary because the other json keys 
> (`l`, `h`, `u`) cannot have whitespace in them.
> 
> I'm not opposed to handing whitespace in an `escapeQuotesAndWhitespace` 
> method though.

I think we should have one that escapes spaces and another that fails on spaces 
to replace the current escapeQuotes.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24202#discussion_r2010551795

Reply via email to