<div dir="ltr"><div><div>Removed the Prefix :)<br><br>2dc1136e2287b52ba3c3c58a1d70de78ad53fef7<br><br></div>Cheers<br><br></div>Tobi<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 22, 2015 at 12:14 PM, Miklos Vajna <span dir="ltr"><<a href="mailto:vmiklos@collabora.co.uk" target="_blank">vmiklos@collabora.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Jan 22, 2015 at 10:58:13AM +0100, Tobias Madl <<a href="mailto:tobias.madl.dev@gmail.com">tobias.madl.dev@gmail.com</a>> wrote:<br>
> But i don't get the thing with changing it to a C++11 enum class. It is<br>
> already an enum class ????<br>
<br>
</span>In your feature branch, you have:<br>
<br>
>  enum class IdlePriority {<br>
<br>
That's a C++11 enum class.<br>
<br>
On master, it's:<br>
<br>
> enum IdlePriority {<br>
<br>
That's why I said -- if you change it to an enum class and you need to<br>
adjust all client code anyway, then I would also drop the now redundant<br>
prefixes as well. See 22f0862ae782c90306d444d2f5aa253b9e97acd2 for<br>
example or 5333782d090a9e147c0c431f0f741863d1d8cf8e.<br>
<br>
Regards,<br>
<br>
Miklos<br>
</blockquote></div><br></div>