Unused local variable in PersistentVector.cons()

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.

Patch: clj-1329-2.patch
Screened by: Alex Miller

Environment

None

Attachments

2
  • 29 Apr 2015, 08:28 PM
  • 22 Jan 2014, 06:08 AM

Activity

Show:

Alex Miller April 29, 2015 at 8:28 PM

Added new patch - same diff but proper format

Andy Fingerhut October 2, 2014 at 12:43 AM

Smit, instructions for creating a patch in the format expected by the screeners is given on the wiki page below. The one you have attached is not in the expected format.

http://dev.clojure.org/display/community/Developing+Patches

Alex Miller March 2, 2014 at 5:37 PM

Thanks Smit!

import March 1, 2014 at 5:51 PM

Comment made by: who

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

import February 2, 2014 at 7:16 PM

Comment made by: who

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

Completed

Details

Assignee

Reporter

Approval

Ok

Patch

Code

Priority

Affects versions

Fix versions

Created January 22, 2014 at 6:08 AM
Updated October 13, 2015 at 3:37 AM
Resolved October 13, 2015 at 3:37 AM