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

You'll need a CSR since you're changing spec text.  (output stream -> 
PrintStream) and the new ImplNote.
It might be worth considering this list() method should be deprecated. 
And/or document as being useful for debugging only.

Consider proposing better replacement.  But that's a bigger (possibly more 
satisfying task)

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

PR Comment: https://git.openjdk.org/jdk/pull/26018#issuecomment-3013459562

Reply via email to