Re: [PATCH] dmapi: Fix a build warning

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Fri, Feb 12, 2010 at 02:56:53PM -0600, Alex Elder wrote:
> Fix a build warning.
> 
> Signed-off-by: Alex Elder <aelder@xxxxxxx>
> 
> ---
>  libdm/getdents.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> Index: dmapi/libdm/getdents.c
> ===================================================================
> --- dmapi.orig/libdm/getdents.c
> +++ dmapi/libdm/getdents.c
> @@ -72,7 +72,9 @@ extern int __have_no_getdents64;
>  # endif
>  #endif
>  
> -#define offsetof(TYPE, MEMBER) ((size_t) &((TYPE *)0)->MEMBER)
> +#ifndef offsetof
> +# define offsetof(TYPE, MEMBER) ((size_t) &((TYPE *)0)->MEMBER)
> +#endif /* ! offsetof */

Looks fine, but as we really should have offsetoff we might just
kill it entirely.

Reviewed-by: Christoph Hellwig <hch@xxxxxx>