Opened at 2009-08-09T22:48:25Z
Closed at 2009-09-21T01:38:47Z
#205 closed fixme (done)
re-binding to another argument doesn't work
| Reported by: | Ichthyostega | Owned by: | Ichthyostega |
|---|---|---|---|
| Priority: | grave | Milestone: | 0integration |
| Component: | lumieraSteam | Keywords: | |
| Sub Tickets: | Parent Tickets: | #144, #148, #266 |
Description
a conceptual error causes re-binding of a command (e.g. to another target object) to break.
Change history (8)
comment:1 by , at 2009-08-09T22:50:35Z
| Status: | new → accepted |
|---|
comment:2 by , at 2009-08-09T23:05:57Z
| blocking: | 144, 148 → 144 |
|---|
(In #148) test is written (and expected behaviour is defined), but still waiting for a preliminary impl. of the registry
comment:3 by , at 2009-08-09T23:08:00Z
| blocking: | 144 → 144, 148 |
|---|
comment:4 by , at 2009-09-07T01:48:19Z
seemingly the key to the solution is not to store a closed functor (thereby overwriting the original functor), but instead to hook into the function invocation operator and use our own CommandClosure object instead of relying on the boost::bind implementation. Details might get a bit more involved, but looks like the better design anyway, so let's just do it...
comment:5 by , at 2009-09-20T00:49:42Z
| blocking: | 144, 148 → 144, 148, 266 |
|---|
comment:6 by , at 2009-09-21T01:38:47Z
| blocking: | 144, 148, 266 → 144, 148 |
|---|---|
| Resolution: | → done |
| Status: | accepted → closed |
broken logic is fixed now...
caused some refactorings, which actually simplified the implementation :)
pending final test (see #148)
comment:7 by , at 2009-10-03T01:33:22Z
| blocking: | 144, 148 → 144, 148, 266 |
|---|
comment:8 by , at 2025-12-25T00:00:00Z
| blocking: | 144, 148, 266 |
|---|---|
| Parent Tickets: | → 144, 148, 266 |
Migration MasterTickets ⟼ Subtickets-plugin

...actually I thought I might get away with a very cheap (in terms of dev time) solution and later on implement it cleanly and without heap allocation. But seemingly I need to do at least part of the work immediately :-/