APersistentVector hasheq method has dead code that might be slowing it down

Description

This is the Java code of the hasheq method for class APersistentVector:


The for loop is dead code, given that the local variable hash is overwritten immediately afterwards. That for loop is also calculating the old hasheq value from before Clojure 1.6, and its value would be incorrect if it were used. Looks like redundant unnecessary code that ought to be deleted.

I have not done any experiments yet to see if eliminating it speeds things up, but seems likely it would.

Environment

None

Assignee

Unassigned

Reporter

Andy Fingerhut

Labels

None

Approval

None

Patch

None

Affects versions

Priority

Minor
Configure