[PATCH 2/3] Add module providing safe DBus operations

Brian C. Lane bcl at redhat.com
Wed Apr 3 22:18:15 UTC 2013


On Wed, Apr 03, 2013 at 04:39:00PM -0400, Chris Lumens wrote:
> > +def dbus_call_safe_sync(service, obj_path, iface, method, args,
> > +                        connection=None):
> > +    """
> > +    Safely all a given method on a given object of a given service over DBus
> 
> Typo - should be "call", not "all".
> 
> > +    @param service: DBus service to use
> > +    @type service: str
> > +    @param obj_path: object path of the object to call method on
> > +    @type obj_path: str
> > +    @param iface: interface to use
> > +    @type iface: str
> > +    @param method: name of the method to call
> > +    @type method: str
> > +    @param args: arguments to pass to the method
> > +    @type args: GVariant
> > +    @param connection: connection to use (if None, a new connection is
> > +                       established)
> > +    @type connection: Gio.DBusConnection
> > +    @return: unpacked value returned by the method
> > +    @rtype: tuple with elements that depend on the method
> > +    @raise DBusCallError: if some DBus related error appears
> > +
> > +    """
> 
> Just curious - what's this new style of docstrings you guys seem to
> like so much?

I was going to ask about that as well. My understanding was that Sphinx
can use either the @param ...: style or :param ...: style, but that the
latter is more popular so that's what I've been trying to add to my new
code.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 482 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/anaconda-patches/attachments/20130403/f1ef8502/attachment.sig>


More information about the anaconda-patches mailing list