I just wasted 2 hours debugging a weird issue due to priv->enabled = enabled; (with enabled as a gboolean) not doing what you'd expect to.
I really don't see the point of using "guint enabled : 1" rather than gboolean. Gaining 7 bits (yeah!) is definitely not worth the extra headache.
Created attachment 50809 [details] [review]
mcd-account: use proper gboolean rather than 1 bit to store boolean properties
It doesn't make sense to not use the boolean type provided by our framework
and using 1 bit variable introduces weird corner case when being casted with
Merged, will be in 5.9.2.