[SOLR-5677] HaversineConstFunction ignores one of its two values, is this on purpose?

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0-ALPHA
• Fix Version/s: 6.0
• Component/s: spatial
• Assignee: Benson Margulies
• Reporter: Benson Margulies
• Created: 07/May/12 00:03
• Updated: 09/May/16 18:59
• Resolved: 30/Jan/14 00:28

Description

org.apache.solr.search.function.distance.HaversineConstFunction.parser.new ValueSourceParser()

{...}

.parse(FunctionQParser)

has an unused variable warning for 'vs2', and uses vs1 to initialize mv2. Maybe vs2 should just be deleted?