Re: [PATCH 6/6] conf: Use consistent function name prefixes for virnwfilterobj

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

 



On 03/06/2017 05:36 PM, John Ferlan wrote:
> Use "virNWFilterObj" as a prefix for any external API in virnwfilterobj
> 
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
> ---
>  src/conf/virnwfilterobj.c      | 34 +++++++++++++++++-----------------
>  src/conf/virnwfilterobj.h      |  6 +++---
>  src/libvirt_private.syms       |  4 ++--
>  src/nwfilter/nwfilter_driver.c |  8 +++-----
>  4 files changed, 25 insertions(+), 27 deletions(-)
> 
> diff --git a/src/conf/virnwfilterobj.c b/src/conf/virnwfilterobj.c
> index 0d81912..7e13afb 100644
> --- a/src/conf/virnwfilterobj.c
> +++ b/src/conf/virnwfilterobj.c
> @@ -117,9 +117,9 @@ virNWFilterObjFindByName(virNWFilterObjListPtr nwfilters,
>  
>  
>  static int
> -_virNWFilterDefLoopDetect(virNWFilterObjListPtr nwfilters,
> -                          virNWFilterDefPtr def,
> -                          const char *filtername)
> +_virNWFilterObjDefLoopDetect(virNWFilterObjListPtr nwfilters,
> +                             virNWFilterDefPtr def,
> +                             const char *filtername)

Ouch. This is even worse than virFunction() + virFunctionInternal()
pair. Moreover, I don't think that we need virNWFilterDefLoopDetect at
all. I mean, this one starting with _ can take its place. The only
difference is that while virNWFilterDefLoopDetect() takes just two
arguments _virNWFilterDefLoopDetect() takes three.

But I guess that can be saved either for a follow up patch or one that's
inserted before this one.

>  {
>      int rc = 0;
>      size_t i;
> @@ -141,8 +141,8 @@ _virNWFilterDefLoopDetect(virNWFilterObjListPtr nwfilters,
>              obj = virNWFilterObjFindByName(nwfilters,
>                                             entry->include->filterref);
>              if (obj) {
> -                rc = _virNWFilterDefLoopDetect(nwfilters,
> -                                               obj->def, filtername);
> +                rc = _virNWFilterObjDefLoopDetect(nwfilters,
> +                                                  obj->def, filtername);
>  
>                  virNWFilterObjUnlock(obj);
>                  if (rc < 0)

Michal

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