On Thu, Jan 22, 2015 at 10:58:13AM +0100, Tobias Madl <[email protected]> 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
signature.asc
Description: Digital signature
_______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
