RFR: 8341566: Add Reader.of(CharSequence) [v7]
Markus KARG
duke at openjdk.org
Thu Oct 10 05:46:12 UTC 2024
On Wed, 9 Oct 2024 21:12:10 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> The test name follows the conventions currently applied in this folder. Feel free to propose a better name, I would be happy to change it.
>
> The test name should provide enough context so the *human* reader does not have to hunt around for the context.
> Either CharSequenceReader or ReaderOfCharSequence would be easier to understand without having to *know* the conventions.
I personally did not have any problem understanding the existing conventions within a second, but OTOH I have no strong feelings here.
@AlanBateman Which of both proposed names shall I use (or shall I keep the current name)?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21371#discussion_r1794696841
More information about the core-libs-dev
mailing list