[
https://issues.apache.org/jira/browse/LUCENE-7205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15239443#comment-15239443
]
Christine Poerschke commented on LUCENE-7205:
---------------------------------------------
I agree, the NodeList.getLength() implementations should be a trivial getter.
With multiple/many child nodes in the node list that would be repeated methods
calls nonetheless, all returning the same result since the list doesn't change,
so this is meant to be very minor optimisation (the extra `int nlLen`
presumably doesn't really cost since even when called within the for-loop the
getLength() result needs to be stored?).
> remove repeated nl.getLength() calls in
> (Boolean|DisjunctionMax|FuzzyLikeThis)QueryBuilder
> ------------------------------------------------------------------------------------------
>
> Key: LUCENE-7205
> URL: https://issues.apache.org/jira/browse/LUCENE-7205
> Project: Lucene - Core
> Issue Type: Task
> Reporter: Christine Poerschke
> Assignee: Christine Poerschke
> Priority: Trivial
> Attachments: LUCENE-7205.patch
>
>
> proposed trivial patch against master to follow
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
