Re: [libvirt PATCH 08/23] util: rename method to virKModIsProhibited

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

 



On Fri, Jun 19, 2020 at 10:32:45 +0100, Daniel Berrange wrote:
> This naming better matches the semantics usage of the test.

I this instance I disagree that it's better as it's diverging from the
terminology used by modprobe and doesn't really describe it any better
than it did before, so I'd prefer if it's not sold as being better.

> Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> ---
>  src/libvirt_private.syms |  2 +-
>  src/util/virfile.c       |  2 +-
>  src/util/virkmod.c       | 24 ++++++++++++------------
>  src/util/virkmod.h       |  2 +-
>  src/util/virpci.c        |  4 ++--
>  5 files changed, 17 insertions(+), 17 deletions(-)

I don't have problem with the change per-se, just the justification for
it.

Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>




[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