Re: [PATCH 03/10] util: storagefile: Introduce helper to free storage source perms

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

 



On 06/19/14 20:57, John Ferlan wrote:
> 
> 
> On 06/19/2014 09:46 AM, Peter Krempa wrote:
>> It will also be reused later.
>> ---
>>  src/util/virstoragefile.c | 16 ++++++++++++----
>>  1 file changed, 12 insertions(+), 4 deletions(-)
>>
> 
> 
> Why is it that patches 1-3 do not update libvirt_private.syms to add new
> src/util/vir* APIs, but some of the future patches do update, such as 4,

The functions added in 1-3 are used only in the utils module, especially
in the storage source deep copy function and are not linked to from
other modules and therefore the symbol export isn't really necessary.

I might add them if you insist, but I don't expect the functions to be
linked to other modules any soon.

> 5, 7?
> 
> John

Peter

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]