On Mon, 30 Jun 2025 06:01:44 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

>> Can I please get a review of this doc-only change which proposes to clarify 
>> the current implementation of the `java.util.Properties.list(...)` methods?
>> 
>> As noted in https://bugs.openjdk.org/browse/JDK-8360575, the current 
>> implementation trims each value to a size of 37 when printing out the value. 
>> This behaviour isn't documented by these methods. The change in this PR adds 
>> an `@implNote` to make a mention of this current behaviour.
>
> Jaikiran Pai has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Alan's review - no need to state the number of characters

src/java.base/share/classes/java/util/Properties.java line 1260:

> 1258:      *      PrintWriter out = ...
> 1259:      *      // list the properties to PrintWriter
> 1260:      *      p.forEach((k, v) -> out.println(k + "=" + v));

Should our example recommend using `toString` as a utility to obtain a debug 
string instead, as it performs no truncation?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26018#discussion_r2203406850

Reply via email to