From mboxrd@z Thu Jan  1 00:00:00 1970
From: Zdenek Kabelac <zkabelac@redhat.com>
Date: Fri, 20 Aug 2010 15:49:29 +0200
Subject: LVM2/lib/report properties.c
In-Reply-To: <20100820130240.24771.qmail@sourceware.org>
References: <20100820130240.24771.qmail@sourceware.org>
Message-ID: <4C6E87E9.9040109@redhat.com>
List-Id: <lvm-devel.redhat.com>
To: lvm-devel@redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

Dne 20.8.2010 15:02, wysochanski at sourceware.org napsal(a):
> CVSROOT:	/cvs/lvm2
> Module name:	LVM2
> Changes by:	wysochanski at sourceware.org	2010-08-20 13:02:39
> 
> Modified files:
> 	lib/report     : properties.c 
> 
> Log message:
> 	Define GET_NUM_PROPERTY_FN macro to simplify numeric property 'get' functions.
> 
> Patches:
> http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/report/properties.c.diff?cvsroot=lvm2&r1=1.2&r2=1.3
> 
> --- LVM2/lib/report/properties.c	2010/08/20 12:45:09	1.2
> +++ LVM2/lib/report/properties.c	2010/08/20 13:02:39	1.3
> @@ -20,6 +20,15 @@
>  #include "lvm-types.h"
>  #include "metadata.h"
>  
> +#define GET_NUM_PROPERTY_FN(NAME, VALUE) \
> +static int _ ## NAME ## _get (void *obj, struct lvm_property_type *prop) \
> +{ \
> +	struct volume_group *vg = (struct volume_group *)obj; \
> +\
> +	prop->v.n_val = VALUE; \
> +	return 1; \
> +}
> +


Is this going to be part of liblvm API - or it's only purpose is to be used
through liblvm2api - in that case it would be better to have this code
compiled as a part of liblvm2api library and not increasing size of liblvm ?

Zdenek