RFR: 8343110: Add getChars(int, int, char[], int) to CharSequence and CharBuffer [v5]
Markus KARG
duke at openjdk.org
Fri Apr 18 12:37:48 UTC 2025
On Wed, 16 Apr 2025 15:00:20 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Yeah, we already added this API to allow implementations to be more consistent under race condition. So for changing char sequences, this method's default implementation can throw IOOBE, but it is already specified so it is well-behaved even under race (as you can argue a read observed the shorter state of the CS that causes the failure)
>
> Omitting it is fine, it was just a reminder that an interface cannot make any definitive statements about its implementations and sometime should warn the unwary user. (We all expect them to adhere to the contract described in each method, but who knows.)
> The statement in the implSpec is fine as it only applies to the default implementation, but can be mis-read and thought to apply to all implementations. YMMV.
> And though I see consensus, I don't see any approvals yet, including the CSR.
> Its good to see it settle down and make progress.
Alan, I am a bit lost here. How to proceed: Change the JavaDocs, or keep them as found in the CSR?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21730#discussion_r2050565025
More information about the nio-dev
mailing list