View Single Post
Posts: 726 | Thanked: 345 times | Joined on Apr 2010 @ Sweden
#7
Originally Posted by nicolai View Post
Yes you are right, but his:
Code:
public:
enum Status
looks like he doesn't want to use it internally only.
In this case it's part of the API but you wrote

enums must be defined in the header file - not the cpp.
which is what commented on.

So, I think that we agree on how it should be done, you just got it a little bit too general in your answer.
 

The Following User Says Thank You to Joorin For This Useful Post: