On Fri, 27 Jun 2025 14:45:19 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:
> 
>   Andy's review suggestions

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

> 1217:      * @implNote If any property's value is greater than 40 characters 
> then
> 1218:      * this method writes only the first 37 characters of that value
> 1219:      * followed by 3 dot characters.

I don't think the note needs to that specific. I think it need only say that 
the implementation truncates the output for very long property values.

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

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

Reply via email to