replace bitfields with flags or separate booleans as appropriate
Submitted by Simon McVittie
Assigned to Telepathy bugs list
Description
This:
- unsigned had_weak_object:1;
is pretty ugly.
Where booleans make most sense, we should use "gboolean had_weak_object;" and just accept the extra memory. We don't necessarily even need to wait for next; we can do this in master.
I suspect that in at least some cases we'd be better off with an explicit flags word, though:
http://blog.ometer.com/2011/01/20/boolean-parameters-are-wrong/(in which case we might be changing the API; if so, that's a reason to make the change in next).
Version: git master