EventManager: Rename queue accessor and remove cast operator
authorAndreas Hansson <andreas.hansson@arm.com>
Mon, 9 Jul 2012 16:35:46 +0000 (12:35 -0400)
committerAndreas Hansson <andreas.hansson@arm.com>
Mon, 9 Jul 2012 16:35:46 +0000 (12:35 -0400)
This patch renames the queue() accessor to the less ambigious
eventQueue, and also removes the cast operator. The queue() member
function cause problems in derived classes that declare members with
the same name, e.g. a MemObject subclass that has a packet queue on
its own. The operator is not causing any harm at this point, but as it
is not used there is little point in keeping it.

src/sim/eventq.hh

index b04b43702e9c48e285c2938a5b9f953f44398597..09483f4e205461eac39547959def8372c295a4aa 100644 (file)
@@ -431,17 +431,12 @@ class EventManager
     EventQueue *eventq;
 
   public:
-    EventManager(EventManager &em) : eventq(em.queue()) {}
-    EventManager(EventManager *em) : eventq(em ? em->queue() : NULL) {}
+    EventManager(EventManager &em) : eventq(em.eventq) {}
+    EventManager(EventManager *em) : eventq(em ? em->eventq : NULL) {}
     EventManager(EventQueue *eq) : eventq(eq) {}
 
     EventQueue *
-    queue() const
-    {
-        return eventq;
-    }
-
-    operator EventQueue *() const
+    eventQueue() const
     {
         return eventq;
     }