Dear all
This practice of putting leading underscores is discouraged these days... ( these are reserved for the library... ) instead we use either trailing underscores ( or m_ prefix... )
So in ThreadImpl class we have
State _state;
instead we should have
State state_;
OR
State m_state;
What do you all think? Just my two cents ;-)
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Dear all
This practice of putting leading underscores is discouraged these days... ( these are reserved for the library... ) instead we use either trailing underscores ( or m_ prefix... )
So in ThreadImpl class we have
State _state;
instead we should have
State state_;
OR
State m_state;
What do you all think? Just my two cents ;-)