Re: [libvirt] [PATCH 03/24] tests: hostdev: Declare count inside CHECK_LIST_COUNT()

> +# define CHECK_LIST_COUNT(list, cnt)                                        \ > +    do {                                                                    \ > +        int count;                                                          \ > +        if ((count = virPCIDeviceListCount(list)) != cnt) {                 \ > +            virReportError(VIR_ERR_INTERNAL_ERROR,                          \ > +                           "Unexpected count of items in " #list ": %d, "   \ > +                           "expecting %zu", count, (size_t) cnt);           \ > +            goto cleanup;                                                   \ > +        }                                                                   \ > +    } while (0)   The only suggestion I would have is to make "count" something less  common, so that you're less likely to end up causing a compiler warning  later if someone adds a variable called "count" to a function that uses  this macro. Otherwise ACK.