On Thu, Jan 22, 2015 at 10:58:13AM +0100, Tobias Madl <tobias.madl.dev@gmail.com> wrote:
But i don't get the thing with changing it to a C++11 enum class. It is already an enum class ????
In your feature branch, you have:
enum class IdlePriority {
That's a C++11 enum class. On master, it's:
enum IdlePriority {
That's why I said -- if you change it to an enum class and you need to adjust all client code anyway, then I would also drop the now redundant prefixes as well. See 22f0862ae782c90306d444d2f5aa253b9e97acd2 for example or 5333782d090a9e147c0c431f0f741863d1d8cf8e. Regards, Miklos
Attachment:
signature.asc
Description: Digital signature