[Bug 40545] New: McdAccount: use gboolean rather than this guint : 1 bullshit

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Sep 1 14:24:28 CEST 2011


https://bugs.freedesktop.org/show_bug.cgi?id=40545

           Summary: McdAccount: use gboolean rather than this guint : 1
                    bullshit
           Product: Telepathy
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: mission-control
        AssignedTo: telepathy-bugs at lists.freedesktop.org
        ReportedBy: guillaume.desmottes at collabora.co.uk
         QAContact: telepathy-bugs at lists.freedesktop.org


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.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.



More information about the telepathy-bugs mailing list