[libvirt PATCH v2 1/1] tests: Introduce virhostdevmock

Andrea Bolognani abologna at redhat.com
Wed May 6 14:27:38 UTC 2020


On Wed, 2020-05-06 at 15:55 +0200, Michal Privoznik wrote:
> On 5/6/20 2:54 PM, Andrea Bolognani wrote:
> > +++ b/tests/virhostdevmock.c
> > @@ -0,0 +1,29 @@
> > +/*
> > + * Copyright (C) 2020 Red Hat, Inc.
> > + * SPDX-License-Identifier: LGPL-2.1-or-later
> > + */
> > +
> > +#include <config.h>
> > +
> > +#include "virutil.h"
> > +#include "virmock.h"
> > +
> > +static char *(*real_virGetUserRuntimeDirectory)(void);
> 
> This is not needed, you are not calling the original function anywhere. 
> Moreover, the original function must be marked as attribute no-inline.

Good point! I've squashed your changes in and pushed. Thanks :)

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list