# HG changeset patch # User Rik # Date 1631058398 25200 # Node ID 598517b1095915b256716e7c2ee6afee4bbd019f # Parent 606652e9f446df5ff9607764732ee56807565379 maint: use "m_" prefix for member variables in class event_queue. event-queue.h: Use "m_" prefix for member variables in class event_queue. diff -r 606652e9f446 -r 598517b10959 libinterp/corefcn/event-queue.h --- a/libinterp/corefcn/event-queue.h Tue Sep 07 16:07:34 2021 -0700 +++ b/libinterp/corefcn/event-queue.h Tue Sep 07 16:46:38 2021 -0700 @@ -40,7 +40,7 @@ { public: - event_queue (void) : fifo () { } + event_queue (void) : m_fifo () { } // No copying! @@ -48,9 +48,8 @@ event_queue& operator = (const event_queue&) = delete; - // Destructor should not raise an exception, so all actions - // registered should be exception-safe. If you're not sure, see - // event_queue_safe. + // Destructor should not raise an exception, so all actions registered + // should be exception-safe. If you're not sure, see event_queue_safe. ~event_queue (void) { run (); } @@ -59,8 +58,8 @@ if (! empty ()) { // No leak on exception! - std::unique_ptr ptr (fifo.front ()); - fifo.pop (); + std::unique_ptr ptr (m_fifo.front ()); + m_fifo.pop (); ptr->run (); } } @@ -69,22 +68,24 @@ { if (! empty ()) { - elem *ptr = fifo.front (); - fifo.pop (); + elem *ptr = m_fifo.front (); + m_fifo.pop (); delete ptr; } } - std::size_t size (void) const { return fifo.size (); } + std::size_t size (void) const { return m_fifo.size (); } protected: void add_action (elem *new_elem) { - fifo.push (new_elem); + m_fifo.push (new_elem); } - std::queue fifo; + //-------- + + std::queue m_fifo; }; // Like event_queue, but this one will guard against the @@ -94,10 +95,6 @@ class event_queue_safe : public event_queue { - private: - - void warn_unhandled_exception (void) const; - public: event_queue_safe (void) : event_queue () { } @@ -122,6 +119,11 @@ } } } + + private: + + void warn_unhandled_exception (void) const; + }; }