RFR: 8266571: Sequenced Collections [v2]

Stuart Marks smarks at openjdk.org
Tue Mar 28 00:04:46 UTC 2023


On Sat, 25 Mar 2023 08:19:18 GMT, Rémi Forax <forax at openjdk.org> wrote:

>> Stuart Marks has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - More specification tweaks.
>>  - Add simple overrides to ArrayList.
>
> This change is absolutely massive, implementing reversed() basically doubles the number of implementations which means multiple years of debugging / spec fixing.
> 
> Reversing a List makes sense, reversing a LinkedHashSet/LinkedHashMap is a nice to have. Having the concept of first and last (getFirst()/getLast()/etc) on Collection is something long awaited. 
> 
> I understand that wanting to separate the concept of Collection and SequencedCollection can be conceptually nice, but 
> multiplying the number of interfaces also multiplies the number of implementations. Pragmatically, this patch is too big compared to how useful it is.
> 
> I get that Oracle is rich, but this JEP is not only a burden for Oracle but for the whole community.

@forax 

Funnily, I was thinking the other day that this change is quite small given that I've been working on it for over two years. :-)

Anyway, thanks for looking through the implementation.

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

PR Comment: https://git.openjdk.org/jdk/pull/7387#issuecomment-1486016843


More information about the core-libs-dev mailing list