EventManager: Remove test for NULL pointer in constructor
authorAndreas Hansson <andreas.hansson@arm.com>
Tue, 21 Aug 2012 09:49:18 +0000 (05:49 -0400)
committerAndreas Hansson <andreas.hansson@arm.com>
Tue, 21 Aug 2012 09:49:18 +0000 (05:49 -0400)
This patch tidies up the EventManager constructor and prunes a corner
case where the EventManager would initialise its eventq pointer to
NULL. This would cause segmentation faults on actual use and should
never happen.

src/sim/eventq.hh

index c4690288990e24d90955f8c9dde7aab27dd62cda..5362dcf34a3055edea6cf03d082b72dc44b1efe7 100644 (file)
@@ -432,7 +432,7 @@ class EventManager
 
   public:
     EventManager(EventManager &em) : eventq(em.eventq) {}
-    EventManager(EventManager *em) : eventq(em ? em->eventq : NULL) {}
+    EventManager(EventManager *em) : eventq(em->eventq) {}
     EventManager(EventQueue *eq) : eventq(eq) {}
 
     EventQueue *