Re: [PATCH v3 07/12] nodedev: Alter node device obj list function names

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

 



On Sat, Jun 03, 2017 at 09:11:57AM -0400, John Ferlan wrote:
> Ensure that any function that walks the node device object list is prefixed
> by virNodeDeviceObjList.
>
> Also, modify the @filter param name for virNodeDeviceObjListExport to
> be @aclfilter.
>
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>

One more from the 'not a fan' patches :)...
ACK

Erik

>  static virNodeDeviceObjPtr
> -virNodeDeviceFindByWWNs(virNodeDeviceObjListPtr devs,
> -                        const char *parent_wwnn,
> -                        const char *parent_wwpn)
> +virNodeDeviceObjListFindByWWNs(virNodeDeviceObjListPtr devs,
> +                               const char *parent_wwnn,
> +                               const char *parent_wwpn)

I assume one would argue that calling this function simply *ByWWN wouldn't
reflect the reality and justify the actual need for both wwnn and wwpn.

--
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]
  Powered by Linux