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

Tomoko Uchida resolved LUCENE-10586.
------------------------------------
    Fix Version/s: 9.3
         Assignee: Tomoko Uchida
       Resolution: Fixed

Thank you both!

> Minor refactoring in Lucene90BlockTreeTermsReader local variables: metaIn, 
> indexMetaIn, termsMetaIn
> ---------------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-10586
>                 URL: https://issues.apache.org/jira/browse/LUCENE-10586
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Tomoko Uchida
>            Assignee: Tomoko Uchida
>            Priority: Trivial
>             Fix For: 9.3
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Those three local variables refer to the same {{IndexInput}} object (no 
> clone() is called).
> {code}
> indexMetaIn = termsMetaIn = metaIn;
> {code}
> I'm not sure but maybe there are some historical reasons. I wonder if it 
> would be better to have only one reference for the underlying {{IndexInput}} 
> object to make it a little easy to follow the code.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to