Clojure

pop! on a 33-item-long transient vector returns a persisted transient vector

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: Release 1.2
  • Component/s: None
  • Labels:
    None
  • Approval:
    Ok

Description

user=> (-> (range 33) vec transient pop! count )
java.lang.IllegalAccessError: Transient used after persistent! call (NO_SOURCE_FILE:0)

Obviously persistent! is never called.

The applied patch doesn't solve a similar issue with transient vectors of size 33+32^n with n >= 2/

Activity

Hide
Assembla Importer added a comment -

cgrand said: [file:aJWSRMAjCr35eseJe5cbLA]: patch

Show
Assembla Importer added a comment - cgrand said: [file:aJWSRMAjCr35eseJe5cbLA]: patch
Hide
Assembla Importer added a comment -

cgrand said: 1.1 has the same bug

Show
Assembla Importer added a comment - cgrand said: 1.1 has the same bug
Hide
Assembla Importer added a comment -
Show
Assembla Importer added a comment - stu said: [file:d-tOhgApGr34XueJe5cbLA]
Hide
Assembla Importer added a comment -

stu said: second patch includes cgrand's fix plus a test

Show
Assembla Importer added a comment - stu said: second patch includes cgrand's fix plus a test
Hide
Assembla Importer added a comment -

cgrand said: (In [[r:7700d66d5a657d2102ddeb560d52bca758669ed9]]) fix pop! on 33-item-long transient vectors. See #358.

Signed-off-by: Stuart Halloway <stu@thinkrelevance.com>
Signed-off-by: STUART HALLOWAY <stuart@STUART-HALLOWAYs-MacBook-Pro.local>

Branch: master

Show
Assembla Importer added a comment - cgrand said: (In [[r:7700d66d5a657d2102ddeb560d52bca758669ed9]]) fix pop! on 33-item-long transient vectors. See #358. Signed-off-by: Stuart Halloway <stu@thinkrelevance.com> Signed-off-by: STUART HALLOWAY <stuart@STUART-HALLOWAYs-MacBook-Pro.local> Branch: master
Hide
Assembla Importer added a comment -

cgrand said: [file:b42sq8AIOr36DmeJe5cbCb]: patch (and updated tests) for transient vectors of size 33+32^n

Show
Assembla Importer added a comment - cgrand said: [file:b42sq8AIOr36DmeJe5cbCb]: patch (and updated tests) for transient vectors of size 33+32^n
Hide
Assembla Importer added a comment -

stu said: Updating tickets (#330, #357, #358, #365)

Show
Assembla Importer added a comment - stu said: Updating tickets (#330, #357, #358, #365)
Hide
Assembla Importer added a comment -

stu said: reopened: cgrand's 5/28 patch needs to be applied.

in general, I would prefer opening a new ticket, and never re-opening old ones. Thoughts?

Show
Assembla Importer added a comment - stu said: reopened: cgrand's 5/28 patch needs to be applied. in general, I would prefer opening a new ticket, and never re-opening old ones. Thoughts?
Hide
Assembla Importer added a comment -

stu said: Updating tickets (#256, #257, #367, #358)

Show
Assembla Importer added a comment - stu said: Updating tickets (#256, #257, #367, #358)

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: