[ 
https://issues.apache.org/jira/browse/LUCENE-5457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13905697#comment-13905697
 ] 

Ryan Ernst commented on LUCENE-5457:
------------------------------------

+1, patch looks good.

{quote}
maybe the cause of my confusion is the local was named badly before?
{quote}
I had asked for the 2*radius in the computation to be moved into the table in 
LUCENE-5271, and the table was renamed, but the variable was missed and I 
didn't notice it.

> Expose SloppyMath earth diameter table
> --------------------------------------
>
>                 Key: LUCENE-5457
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5457
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Adrien Grand
>            Assignee: Adrien Grand
>            Priority: Trivial
>         Attachments: LUCENE-5457.patch
>
>
> LUCENE-5271 introduced a table in order to get approximate values of the 
> diameter of the earth given a latitude. This could be useful for other 
> computations so I think it would be nice to have a method that exposes this 
> table.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

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

Reply via email to