Code review comment for lp:~andreas-pokorny/mir/add-timer-to-main-loop

79 + enum State
80 + {
81 + Pending, /**< Will trigger the callback at some point in the future */
82 + Cancelled, /**< The callback has been cancelled before being triggered */
83 + Triggered /**< The callback has been called */
84 + };

Should be "pending", "cancelled" & "triggered"

http://unity.ubuntu.com/mir/cppguide/index.html?showone=Enumerator_Names#Enumerator_Names