ClojureScript

(persistent! (assoc! (transient [0]) nil 1)) => [1]

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    r2156
  • Patch:
    Code and Test

Description

assoc!-ing a non-numeric index into a transient vector is the same as assoc!-ing index 0. This should throw an exception instead. Patch and tests attached.

  1. cljs-768.patch
    21/Feb/14 9:31 AM
    2 kB
    Francis Avila
  2. cljs-768.patch
    17/Feb/14 11:39 AM
    1 kB
    Francis Avila

Activity

Hide
David Nolen added a comment -

This patch no does not cleanly apply on master. Can we get a new patch? Thanks.

Show
David Nolen added a comment - This patch no does not cleanly apply on master. Can we get a new patch? Thanks.
Hide
Francis Avila added a comment -

Rebased patch.

Show
Francis Avila added a comment - Rebased patch.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: