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

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

 



On Fri, Mar 07, 2014 at 06:53:09PM +0800, Chunyan Liu wrote:
> Signed-off-by: Chunyan Liu <cyliu@xxxxxxxx>
> ---
>  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 :|

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