@@ -694,6 +694,8 @@ module Roby return if @unreachable @unreachable = true + EventGenerator.event_gathering.delete(self) + unreachable_handlers.each do |_, block| begin block.call(reason) lib/roby/event.rb @@ -772,6 +772,7 @@ class TC_Event < Test::Unit::TestCase # removed from the plan collection.clear plan.remove_object(e1) + assert(!EventGenerator.event_gathering.has_key?(e1)) EventGenerator.remove_event_gathering(collection) end test/test_event.rb a350e81f184577df8d946dcd451bf204e8b9dfbd Sylvain Joyeux sylvain.joyeux@dfki.de http://github.com/doudou/roby/commit/0b777b8f4b494c7e88fa83d2d8fe683dbd1b34dc 0b777b8f4b494c7e88fa83d2d8fe683dbd1b34dc 2008-06-04T08:58:10-07:00 2008-06-04T08:58:10-07:00 [core] fix bug and memory leak related to EventGenerator.gather_events .gather_events has been designed to have the propagation code push the set of fired events in collections, so that plan analysis code can avoid iterating on the whole plan to find the interesting events. Until now, the automatic de-registration of the events when they become unreachable was actually not done, resulting in a memory leak. This commit fixes it -- and updates the test suite so that it does not appear anymore. 629c5b4c071a53302632134ec6790dc7e16d6dd6 Sylvain Joyeux sylvain.joyeux@dfki.de