[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-dbus
Subject:    Re: dbus_connection_send_with_reply_and_block and errors
From:       Havoc Pennington <hp () redhat ! com>
Date:       2005-08-09 20:31:14
Message-ID: 1123619474.27091.42.camel () localhost ! localdomain
[Download RAW message or body]

Hi,

Thanks, this looks good, missing a couple spaces-before-parens:

On Tue, 2005-08-09 at 21:58 +0300, Timo Teräs wrote:
> +  real->name = _dbus_strdup(name);

space

> +      dbus_free(real->message);

space
 
> -  _dbus_string_free (&str);

Sure you should remove this?

Havoc



[prev in list] [next in list] [prev in thread] [next in thread]