RFR: 8341566: Add Reader.of(CharSequence) [v10]

[Chen Liang]

On Fri, 11 Oct 2024 21:02:30 GMT, Markus KARG <duke at openjdk.org> wrote:

>> src/java.base/share/classes/java/io/Reader.java line 212:
>> 
>>> 210:                     default -> {
>>> 211:                         for (int i = next, j = next + n; i < j;)
>>> 212:                             cbuf[off++] = cs.charAt(i++);
>> 
>> The single increment is fine and readable. 
>> Introducing a local variable for the end (j) seems unnecessary and its name is not indicative of its purpose as a limit.
>
> Uhm... actually that is a quite common pattern we even teach people everyday, and in the past 30 years none of them had a problem understanding it... 🤔  Anyways, really I don't see any benefit in this whole discussion, as I said, I have no feelings about that tiny part of this PR at all. Simply post the exact code you want to have and I am fine to put it into place! 😃

I would say this current version is fine: that's how we handle encode/decode loop in classes like `String`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21371#discussion_r1797478583