Re: [PATCH 3/5] virdbus: Make virDBusCall static

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/05/2014 02:53 AM, Guido Günther wrote:
> On Sat, May 03, 2014 at 03:59:40PM -0400, Cole Robinson wrote:
>> ---
>>  src/libvirt_private.syms |  1 -
>>  src/util/virdbus.c       | 19 +++++--------------
>>  src/util/virdbus.h       |  4 ----
>>  3 files changed, 5 insertions(+), 19 deletions(-)
>>
>> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
>> index cc3ae2c..8a015eb 100644
>> --- a/src/libvirt_private.syms
>> +++ b/src/libvirt_private.syms
>> @@ -1148,7 +1148,6 @@ virCryptoHashString;
>>  
>>  
>>  # util/virdbus.h
>> -virDBusCall;
>>  virDBusCallMethod;
>>  virDBusCloseSystemBus;
>>  virDBusCreateMethod;
>> diff --git a/src/util/virdbus.c b/src/util/virdbus.c
>> index 4217aea..709d6ee 100644
>> --- a/src/util/virdbus.c
>> +++ b/src/util/virdbus.c
>> @@ -1404,10 +1404,11 @@ int virDBusCreateReply(DBusMessage **reply,
>>   *
>>   * Returns 0 on success, or -1 upon error
>>   */
>> -int virDBusCall(DBusConnection *conn,
>> -                DBusMessage *call,
>> -                DBusMessage **replyout,
>> -                DBusError *error)
>> +static int
>> +virDBusCall(DBusConnection *conn,
>> +            DBusMessage *call,
>> +            DBusMessage **replyout,
>> +            DBusError *error)
>>  {
>>      DBusMessage *reply = NULL;
>>      DBusError localerror;
>> @@ -1694,16 +1695,6 @@ int virDBusCreateReply(DBusMessage **reply ATTRIBUTE_UNUSED,
>>      return -1;
>>  }
>>  
>> -int virDBusCall(DBusConnection *conn ATTRIBUTE_UNUSED,
>> -                DBusMessage *call ATTRIBUTE_UNUSED,
>> -                DBusMessage **reply ATTRIBUTE_UNUSED,
>> -                DBusError *error ATTRIBUTE_UNUSED)
>> -{
>> -    virReportError(VIR_ERR_INTERNAL_ERROR,
>> -                   "%s", _("DBus support not compiled into this binary"));
>> -    return -1;
>> -}
>> -
>>  int virDBusCallMethod(DBusConnection *conn ATTRIBUTE_UNUSED,
>>                        DBusMessage **reply ATTRIBUTE_UNUSED,
>>                        DBusError *error ATTRIBUTE_UNUSED,
>> diff --git a/src/util/virdbus.h b/src/util/virdbus.h
>> index a4987b8..d0c7de2 100644
>> --- a/src/util/virdbus.h
>> +++ b/src/util/virdbus.h
>> @@ -68,10 +68,6 @@ int virDBusCallMethod(DBusConnection *conn,
>>                        const char *iface,
>>                        const char *member,
>>                        const char *types, ...);
>> -int virDBusCall(DBusConnection *conn,
>> -                DBusMessage *call,
>> -                DBusMessage **reply,
>> -                DBusError *error);
>>  int virDBusMessageRead(DBusMessage *msg,
>>                         const char *types, ...);
>>  void virDBusMessageUnref(DBusMessage *msg);
>> -- 
> 
> Ack. Outside consumers use the higher level virDBusCallMethod.
>  -- Guido
> 

Thanks, I pushed this

- Cole

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]