Bug 54 - add/remove listener mechanism broken
Status:
RESOLVED FIXED
Component:
Core
Version:
unspecified
Hardware:
Other Other
Importance:
P3 normal
Target Milestone:
---
Assignee:
Rob Caelers
URL:
Depends on:
Blocks:
Reported:
Sep 9 2002 23:03:45 UTC
by:
Raymond Penners
Modified:
Sep 11 2002 20:42:25 UTC
WhoWhenWhatRemovedAdded
Rob CaelersSep 10 2002 21:14:12 UTCstatusNEWASSIGNED
Rob CaelersSep 11 2002 20:42:25 UTCstatusASSIGNEDRESOLVED
resolutionFIXED
Description
Raymond Penners  Sep 9 2002 23:03:45 UTC
Currently, the API is the following:

add_listener(key_prefix, listener);
remove_listener(key_prefix);

The latter method should become:

remove_listener(listener);

Rationale:
- the current API could result in race conditions if two independent parts of the program both listen to events

While you're at it, add some missing remove_listener() calls (e.g. in the MainWindow)
Comment 1
Rob Caelers  Sep 11 2002 20:42:25 UTC
not fully tested.....