ClojureScript

subvec should avoid creating layered Subvecs

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

cljs.core/subvec (or rather build-subvec, the implementation detail behind it) currently fails to check whether its vector argument is already a Subvec and, if so, base the new Subvec off the original regular vector. The Clojure implementation does perform the unwrapping.

The forthcoming patch fixes this and includes a basic test.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: