From mboxrd@z Thu Jan  1 00:00:00 1970
From: Zdenek Kabelac <zkabelac@redhat.com>
Date: Fri, 12 Dec 2008 09:26:57 -0000
Subject: Re: LVM2 ./WHATS_NEW_DM libdm/libdm-deptree.c
In-Reply-To: <1229013323.20795.3.camel@localhost.localdomain>
References: <20081211162552.29826.qmail@sourceware.org>
	<1229013323.20795.3.camel@localhost.localdomain>
Message-ID: <49422E5E.7030700@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

Dave Wysochanski napsal(a):
>> +/* simplify string emiting code */
>> +#define EMIT_PARAMS(p, str...)\
>> +	do {\
>> +		const size_t bufsize = paramsize - (size_t)p;\
>> +		int w;\
>> +		\
>> +		if ((w = snprintf(params + p, bufsize, str)) < 0\
>> +		    || ((size_t)w >= bufsize)) {\
>> +			stack; /* Out of space */\
>> +			return -1;\
>> +		}\
>> +		p += w;\
>> +	} while (0)
>> +
> 
> Do we have to do a macro here?  Macros like this are harder to debug...
> 


I think it's actually minimizing the chance you will add a buggy code by some 
cut&paste operation when you add new string emitting line.

Also it makes the code more readable.

Do you think there is some potential debug problem in this code, that makes
worth to keep the original emitting style?

(i.e. replicator emits string 11 times - without this macro it makes the code 
even less readable,  and also using inline function and 'stack' macro is not 
going to work together, so it's hard to use gdb-friendly inline here :( )

Zdenek