From 4fdf3c1b77349c8129c408483bac5623d61963ad Mon Sep 17 00:00:00 2001
From: Rudi Grinberg <rudi.grinberg@sourcefabric.org>
Date: Wed, 5 Sep 2012 16:04:11 -0400
Subject: [PATCH] MM2: Removed assertion in favor of warning.
---
.../media-monitor2/media/monitor/eventcontractor.py | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/python_apps/media-monitor2/media/monitor/eventcontractor.py b/python_apps/media-monitor2/media/monitor/eventcontractor.py
index f9ff96dd7..ececfb846 100644
--- a/python_apps/media-monitor2/media/monitor/eventcontractor.py
+++ b/python_apps/media-monitor2/media/monitor/eventcontractor.py
@@ -48,8 +48,12 @@ class EventContractor(Loggable):
# checked against the newest event 'evt' in this case
self.unregister( old_e )
evt.add_safe_pack_hook( lambda : self.__unregister(evt) )
- assert evt.path not in self.store, \
- "Clean up should have been called by '%s'" % evt
+
+ if evt.path in self.store:
+ self.logger.warn("Clean up should have been called by '%s'" %
+ str(evt))
+ self.logger.warn("Overwriting event for '%s'" % evt.path)
+
self.store[ evt.path ] = evt
return True # We actually added something, hence we return true.