On 04/22/2014 06:49 AM, Jiri Denemark wrote: > Returning both virStorageSourcePtr and its path member does not make a > lot of sense. Yep - artifacts from pre-refactoring that are now pointless. > > Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx> > --- > src/qemu/qemu_driver.c | 7 +++---- > src/util/virstoragefile.c | 16 +++++++--------- > src/util/virstoragefile.h | 7 +++---- > tests/virstoragetest.c | 39 +++++++++++++++++++++++---------------- > 4 files changed, 36 insertions(+), 33 deletions(-) ACK with one nit: > +++ b/src/util/virstoragefile.c > @@ -1510,9 +1510,9 @@ int virStorageFileGetSCSIKey(const char *path, > * Since the results point within CHAIN, they must not be > * independently freed. Reports an error and returns NULL if NAME is > * not found. */ > -const char * > +virStorageSourcePtr The comments prior to this function are now out of date. Update them to something like: Given a CHAIN, look for the backing file NAME within the chain and return that location within the chain. Pass NULL for NAME to find the base of the chain. If PARENT is not NULL, set *PARENT to the parent of the result (or to NULL if NAME matches the start of the chain). Reports an error and returns NULL if NAME is not found. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list