RFR: 8263561: Re-examine uses of LinkedList [v5]

Claes Redestad redestad at openjdk.java.net
Mon Aug 2 11:40:36 UTC 2021


On Mon, 26 Jul 2021 08:27:20 GMT, Сергей Цыпанов <github.com+10835776+stsypanov at openjdk.org> wrote:

>> After I've renamed remove branch GitHub for some reason has closed original https://github.com/openjdk/jdk/pull/2744, so I've decided to recreate it.
>
> Сергей Цыпанов has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
> 
>  - Merge branch 'master' into 8263561
>  - Merge branch 'master' into 8263561
>  - Merge branch 'master' into 8263561
>  - Merge branch 'master' into 8263561
>  - Merge branch 'master' into 8263561
>    
>    # Conflicts:
>    #	src/java.base/unix/classes/sun/net/dns/ResolverConfigurationImpl.java
>  - Merge branch 'master' into purge-linked-list
>  - 8263561: Use sized constructor where reasonable
>  - 8263561: Use interface List instead of particular type where possible
>  - 8263561: Rename requestList -> requests
>  - 8263561: Re-examine uses of LinkedList

I always approve of removing LinkedLists and Vectors.

Using ArrayDeque in AbstractPoller seems like the right choice.

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

Marked as reviewed by redestad (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4304


More information about the nio-dev mailing list