Clojure

Unused local variable in PersistentVector.cons()

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Trivial Trivial
  • Resolution: Unresolved
  • Affects Version/s: Release 1.5, Release 1.6
  • Fix Version/s: None
  • Component/s: None
  • Labels:
  • Patch:
    Code
  • Approval:
    Triaged

Description

in src/jvm/clojure/lang/PersistentVector.java:168, there is an integer i being defined which is not being used anywhere in the method.

https://github.com/clojure/clojure/blob/master/src/jvm/clojure/lang/PersistentVector.java#L168

Activity

Hide
Alex Miller added a comment -

Thanks Smit!

Show
Alex Miller added a comment - Thanks Smit!
Hide
Smit Shah added a comment -

Stuart, I have successfully submitted the CA (http://clojure.org/contributing).
I guess now merging this patch shouldn't be a problem

Show
Smit Shah added a comment - Stuart, I have successfully submitted the CA (http://clojure.org/contributing). I guess now merging this patch shouldn't be a problem
Hide
Smit Shah added a comment -

Stuart, I will send the CA via post ASAP.
It might take a couple of days to reach Rich though.

Show
Smit Shah added a comment - Stuart, I will send the CA via post ASAP. It might take a couple of days to reach Rich though.
Hide
Stuart Halloway added a comment -

Smit, can you please submit a CA? http://clojure.org/contributing

Show
Stuart Halloway added a comment - Smit, can you please submit a CA? http://clojure.org/contributing

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated: