RFR: 8353795: Add Writer.of(StringBuilder) [v2]

Markus KARG duke at openjdk.org
Sun May 4 16:21:42 UTC 2025


On Sun, 4 May 2025 14:52:39 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Markus KARG has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Undone copyright update of otherwise unchanged file.
>>  - Update Of.java
>>    
>>    Applied changnes proposed by @liach: "the default toString already includes id=..., so I usually don't provide an explicit override to make the code concise."
>
> src/java.base/share/classes/java/nio/X-Buffer.java.template line 2:
> 
>> 1: /*
>> 2:  * Copyright (c) 2000, 2025, Oracle and/or its affiliates. All rights reserved.
> 
> Redundant change.

Good catch! Undone copyright update in https://github.com/openjdk/jdk/pull/24469/commits/ea435ed1090d8f028262f5900baf0a00472726d9.

> test/jdk/java/io/Writer/Of.java line 45:
> 
>> 43:         @Override
>> 44:         public String toString() {
>> 45:             return id; // allows to identify config when test case fails
> 
> I used the same strategy in junit tests; the default toString already includes id=..., so I usually don't provide an explicit override to make the code concise.

Understood. Removed override in https://github.com/openjdk/jdk/pull/24469/commits/f48b99995723b6c4dd73014015f50b1047ff28cd.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24469#discussion_r2072655279
PR Review Comment: https://git.openjdk.org/jdk/pull/24469#discussion_r2072654985


More information about the core-libs-dev mailing list