Clojure

Eliminate many uses of reflection in Clojure code

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: Release 1.4, Release 1.5
  • Fix Version/s: None
  • Component/s: None
  • Patch:
    Code

Description

There are dozens of uses of reflection in Clojure code that can be eliminated by adding suitable type hints. This patch adds the necessary type hints for most of those.

Activity

Hide
Andy Fingerhut added a comment -

Patch clj-1080-eliminate-many-reflection-warnings-patch-v1.txt dated Sep 30 2012 adds type hints to eliminate many uses of reflection in Clojure core code.

Show
Andy Fingerhut added a comment - Patch clj-1080-eliminate-many-reflection-warnings-patch-v1.txt dated Sep 30 2012 adds type hints to eliminate many uses of reflection in Clojure core code.
Andy Fingerhut made changes -
Field Original Value New Value
Attachment clj-1080-eliminate-many-reflection-warnings-patch-v1.txt [ 11534 ]
Andy Fingerhut made changes -
Patch Code [ 10001 ]
Hide
Andy Fingerhut added a comment -

clj-1080-eliminate-many-reflection-warnings-patch-v2.txt dated Nov 14 2012 is identical to the previous patch (to be removed soon), except it applies cleanly to latest master.

Show
Andy Fingerhut added a comment - clj-1080-eliminate-many-reflection-warnings-patch-v2.txt dated Nov 14 2012 is identical to the previous patch (to be removed soon), except it applies cleanly to latest master.
Andy Fingerhut made changes -
Attachment clj-1080-eliminate-many-reflection-warnings-patch-v2.txt [ 11677 ]
Andy Fingerhut made changes -
Attachment clj-1080-eliminate-many-reflection-warnings-patch-v1.txt [ 11534 ]
Hide
Andy Fingerhut added a comment -

clj-1080-eliminate-many-reflection-warnings-patch-v3.txt dated Feb 7 2013 is identical to the previous patch (to be removed soon), except it applies cleanly to latest master. One type hint in the patch was added due to a different change, and was no longer needed in the patch.

Show
Andy Fingerhut added a comment - clj-1080-eliminate-many-reflection-warnings-patch-v3.txt dated Feb 7 2013 is identical to the previous patch (to be removed soon), except it applies cleanly to latest master. One type hint in the patch was added due to a different change, and was no longer needed in the patch.
Andy Fingerhut made changes -
Attachment clj-1080-eliminate-many-reflection-warnings-patch-v3.txt [ 11841 ]
Andy Fingerhut made changes -
Attachment clj-1080-eliminate-many-reflection-warnings-patch-v2.txt [ 11677 ]
Alex Miller made changes -
Labels performance typehints
Hide
Andy Fingerhut added a comment -

Patch clj-1080-v4.txt eliminates many, but not all, uses of reflection. To avoid overlap with CLJ-1259, it does not touch pprint or any of the files loaded from pprint.clj. See CLJ-1259 for those.

Show
Andy Fingerhut added a comment - Patch clj-1080-v4.txt eliminates many, but not all, uses of reflection. To avoid overlap with CLJ-1259, it does not touch pprint or any of the files loaded from pprint.clj. See CLJ-1259 for those.
Andy Fingerhut made changes -
Attachment clj-1080-v4.txt [ 12251 ]
Andy Fingerhut made changes -
Attachment clj-1080-eliminate-many-reflection-warnings-patch-v3.txt [ 11841 ]
Hide
Andy Fingerhut added a comment -

Patch clj-1080-v5.txt eliminates many, but not all, uses of reflection. It does not touch pprint or any of the files loaded from pprint.clj – see CLJ-1259 for those. Similarly see CLJ-1277 for elimination of reflection in instant.clj

Show
Andy Fingerhut added a comment - Patch clj-1080-v5.txt eliminates many, but not all, uses of reflection. It does not touch pprint or any of the files loaded from pprint.clj – see CLJ-1259 for those. Similarly see CLJ-1277 for elimination of reflection in instant.clj
Andy Fingerhut made changes -
Attachment clj-1080-v5.txt [ 12310 ]
Andy Fingerhut made changes -
Attachment clj-1080-v4.txt [ 12251 ]

People

Vote (0)
Watch (3)

Dates

  • Created:
    Updated: