[libvirt] [dbus RFC 01/11] util: introduce VIRT_ARRAY_CARDINALITY macro
Pavel Hrdina
phrdina at redhat.com
Tue Jan 23 10:18:44 UTC 2018
On Tue, Jan 23, 2018 at 09:48:51AM +0000, Daniel P. Berrange wrote:
> On Mon, Jan 22, 2018 at 06:15:59PM +0100, Pavel Hrdina wrote:
> > Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> > ---
> > src/util.h | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/src/util.h b/src/util.h
> > index 0c9f3d7..6d99841 100644
> > --- a/src/util.h
> > +++ b/src/util.h
> > @@ -8,6 +8,8 @@
> > #define _cleanup_(_x) __attribute__((__cleanup__(_x)))
> > #define VIR_ATTR_UNUSED __attribute__((__unused__))
> >
> > +#define VIRT_ARRAY_CARDINALITY(array) (sizeof(array) / sizeof(*(array)))
> > +
> >
> > int
> > virtDBusUtilMessageAppendTypedParameters(sd_bus_message *message,
>
> Reviewed-by: Daniel P. Berrange <berrange at redhat.com>
>
> Though if I was naming this again in libvirt I would follow glib and call
> it VIR_N_ELEMENTS as it way shorter & easier to spell than the word
> CARDINALITY :-) Your call !
That's definitely better name, I'll use that one, thanks.
Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180123/d3bf2aa8/attachment-0001.sig>
More information about the libvir-list
mailing list