[libvirt] [PATCH v14 42/49] rename some function names to keep consistency

Daniel P. Berrange berrange at redhat.com
Wed Mar 12 15:12:46 UTC 2014


On Fri, Mar 07, 2014 at 06:53:09PM +0800, Chunyan Liu wrote:
> Signed-off-by: Chunyan Liu <cyliu at suse.com>
> ---
>  src/libvirt_private.syms |  8 ++++----
>  src/qemu/qemu_hostdev.c  | 16 +++++++--------
>  src/util/virhostdev.c    | 52 ++++++++++++++++++++++++------------------------
>  src/util/virhostdev.h    | 40 ++++++++++++++++++-------------------
>  4 files changed, 58 insertions(+), 58 deletions(-)
> 
> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
> index 5bb529c..e7d9d56 100644
> --- a/src/libvirt_private.syms
> +++ b/src/libvirt_private.syms
> @@ -1297,10 +1297,10 @@ virHostdevManagerGetDefault;
>  virHostdevPciNodeDeviceDetach;
>  virHostdevPciNodeDeviceReAttach;
>  virHostdevPciNodeDeviceReset;
> -virHostdevPreparePCIDevices;
> -virHostdevPrepareSCSIDevices;
> -virHostdevPrepareUSBDevices;
> -virHostdevReAttachPCIDevices;
> +virHostdevPreparePciHostdevs;
> +virHostdevPrepareScsiHostdevs;
> +virHostdevPrepareUsbHostdevs;
> +virHostdevReAttachPciHostdevs;
>  virHostdevReAttachScsiHostdevs;
>  virHostdevReAttachUsbHostdevs;
>  virHostdevUpdateActivePciHostdevs;

IMHO this is fixing the wrong side of the inconsistency.

PCI, USB and SCSI are all abbreviations, so using capitalization
is correct. We should instead fix the Pci, Usb and Scsi names.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the libvir-list mailing list