sends from agent error-handlers should be allowed
Description
Environment
Attachments
- 08 Nov 2010, 06:54 AM
Activity
chouser November 8, 2010 at 6:54 AM
The test added previously was failing because *agent* wasn't bound in the error handler, per CLJ-672.
This patch (0003-) subsumes the previous patches and adds a similar test that does not rely on *agent* being bound. This would have succeeded all along. After a fix for CLJ-672, both tests pass.
Setting this back to Test, but it will fail unless tested after applying the patch from CLJ-672.
chouser November 7, 2010 at 11:29 PM
Test appears to failing because of https://clojure.atlassian.net/browse/CLJ-672#icft=CLJ-672
Stuart Halloway November 5, 2010 at 5:06 PM
Heisenbugs! Test no longer passing, don't have time to look at it atm. Moving back from OK to Test.
import August 24, 2010 at 5:57 PM
Comment made by: importer
chouser@n01se.net said: [[file:d9EEI0G4yr36YneJe5cbLr]]: Set nested to null instead of to empty vector before invoking error handler
import August 24, 2010 at 5:57 PM
Comment made by: importer
Converted from http://www.assembla.com/spaces/clojure/tickets/390
Attachments:
0001-Allows-agent-error-handler-to-send-successfully.-Ref.patch - https://www.assembla.com/spaces/clojure/documents/d9EEI0G4yr36YneJe5cbLr/download/d9EEI0G4yr36YneJe5cbLr
0390-agent-error-send-plus-test.patch - https://www.assembla.com/spaces/clojure/documents/cVNaMc15Wr35VzeJe5cbLr/download/cVNaMc15Wr35VzeJe5cbLr
Details
Assignee
chouserchouserReporter
chouserchouserApproval
OkPatch
Code and TestPriority
MajorFix versions
Details
Details
Assignee
Reporter
Approval
Patch
Priority
Major
Currently if an agent's error-handler sends to any agent, the sent action is
silently thrown away. The send should instead be allowed.
Originally reported here:
http://groups.google.com/group/clojure/browse_frm/thread/c1d05bdbb50d7d28