[libvirt] [PATCH 04/18] virpcimock: Drop needless typecast
Daniel Henrique Barboza
danielhb413 at gmail.com
Fri Aug 16 21:06:56 UTC 2019
Reviewed-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
On 8/14/19 8:57 AM, Michal Privoznik wrote:
> When creating a PCI device, the pciDevice structure contains @id
> member which holds device address (DDDD.BB:DD.F) and is type of
> 'char *'. But the structure is initialized from a const char and
> in fact we never modify or free the @id.
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
> tests/virpcimock.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/virpcimock.c b/tests/virpcimock.c
> index 1c21e4e045..853ac588e9 100644
> --- a/tests/virpcimock.c
> +++ b/tests/virpcimock.c
> @@ -120,7 +120,7 @@ struct pciDriver {
> };
>
> struct pciDevice {
> - char *id;
> + const char *id;
> int vendor;
> int device;
> int klass;
> @@ -878,7 +878,7 @@ init_env(void)
>
> # define MAKE_PCI_DEVICE(Id, Vendor, Device, ...) \
> do { \
> - struct pciDevice dev = {.id = (char *)Id, .vendor = Vendor, \
> + struct pciDevice dev = {.id = Id, .vendor = Vendor, \
> .device = Device, __VA_ARGS__}; \
> pci_device_new_from_stub(&dev); \
> } while (0)
More information about the libvir-list
mailing list