[Telepathy] [Bug 18091] New: include IMPLEMENT(foo) boilerplate in interface documentation

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Oct 16 06:05:48 PDT 2008


http://bugs.freedesktop.org/show_bug.cgi?id=18091

           Summary: include IMPLEMENT(foo) boilerplate in interface
                    documentation
           Product: Telepathy
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: low
         Component: telepathy-glib
        AssignedTo: telepathy at lists.freedesktop.org
        ReportedBy: will.thompson at collabora.co.uk


Just about every implementation of every D-Bus interface in every CM uses this
idiom:

#define IMPLEMENT(x) tp_svc_channel_implement_##x (\
    klass, gabble_im_channel_##x)
  IMPLEMENT(close);
  IMPLEMENT(get_channel_type);
  IMPLEMENT(get_handle);
  IMPLEMENT(get_interfaces);
#undef IMPLEMENT

I always end up cargo-culting this, and it's easy to get wrong, particularly if
I accidentally copy from somewhere that reimplements most but not all of the
methods.

It'd be nice to have this boilerplate in the documentation of the interface
type, as a canonical source.


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


More information about the Telepathy mailing list