Re: [PATCH 1/1] support libnl-3 (v2)

[Serge Hallyn <serge.hallyn@xxxxxxxxxxxxx>]

Quoting Eric Blake (eblake@xxxxxxxxxx):
> On 05/03/2012 11:55 AM, Stefan Berger wrote:
> >>
> >> +#ifdef HAVE_LIBNL1
> >> +#define nl_alloc nl_handle_alloc
> >> +#define nl_free nl_handle_destroy
> >> +typedef struct nl_handle nlhandle_t;
> >> +#else
> >> +#define nl_alloc nl_socket_alloc
> >> +#define nl_free nl_socket_free
> >> +typedef struct nl_sock nlhandle_t;
> >> +#endif
> >> +
> > 
> > I would not #define in the namespace of that library (nl_*).
> 
> Agreed that a vir* namespace is safer.
> 
> > What about
> > the following:
> 
> > #ifdef HAVE_LIBNL1
> > 
> > static struct nl_handle *
> > virNLHandleAlloc(void)
> > {
> >     return nl_handle_alloc();
> > }
> 
> One further:
> 
> typedef struct nl_handle virNLHandle;
> 
> static virNLHandle *
> virNLHandleAlloc(void) ...
> 
> so that the rest of the code is indeed isolated into virNL wrappers with
> no additional #ifdefs.

Yup, I like it, thanks guys.

I don't know whether I'll have time to send a new patch tomorrow.  If not
I'll aim to write one over the weekend, but if someone else wants to make
the (somewhat trivial) updates I won't feel upstaged :)

thanks,
-serge

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list