[ 
https://issues.apache.org/jira/browse/LUCENE-6831?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dawid Weiss updated LUCENE-6831:
--------------------------------
    Description: 
I quickly scanned the code (out of curiosity) and most of the use cases of 
LinkedList are as a Queue, in which case indeed an ArrayDeque would be a better 
choice, especially if the maximum size is known in advance.

There are also some invalid/ incorrect uses like calling size() on a linked 
list in {{MultiPhraseQueryNodeBuilder}}, which should be fixed.

> Review LinkedList usage
> -----------------------
>
>                 Key: LUCENE-6831
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6831
>             Project: Lucene - Core
>          Issue Type: Task
>            Reporter: Dawid Weiss
>            Assignee: Dawid Weiss
>            Priority: Trivial
>             Fix For: Trunk
>
>
> I quickly scanned the code (out of curiosity) and most of the use cases of 
> LinkedList are as a Queue, in which case indeed an ArrayDeque would be a 
> better choice, especially if the maximum size is known in advance.
> There are also some invalid/ incorrect uses like calling size() on a linked 
> list in {{MultiPhraseQueryNodeBuilder}}, which should be fixed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to