On Tue, 12 May 2026 10:37:12 GMT, Per Minborg <[email protected]> wrote:

>> This PR proposes to add a new `default` method `List::removeAtIndex`.
>> 
>> There are two overloads of the method `List::remove`, and if the list is of 
>> type `List<Integer>`, the overload resolution could pick a surprising 
>> variant.
>> 
>> Hence, it is better to add a separate method that removes an element based 
>> on its index.
>> 
>> It is proposed that the `E remove(int index)` method is _not_ `@Deprecated`. 
>> Instead, we add verbiage to promote the new method over the old one.
>> 
>> ---------
>> - [X] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Per Minborg has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains three additional 
> commits since the last revision:
> 
>  - Merge branch 'master' into rfe-list-remove-at-index
>  - Add @apiNote
>  - Add List::removeAtIndex

I have bumped this PR to JDK 28, so we have plenty of time to discuss.

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

PR Comment: https://git.openjdk.org/jdk/pull/31064#issuecomment-4440648800

Reply via email to