[Debburn-devel] [Debburn-changes] r787 - in cdrkit/trunk: genisoimage genisoimage/diag icedax include libhfs_iso librols libusal libusal/usal wodim

[Harald Hoyer]

Peter Samuelson wrote:
> [Harald Hoyer]
>>>> @@ -58,8 +58,8 @@
>>>>  #ifndef	STDERR_FILENO
>>>>  #define	STDERR_FILENO	2
>>>>  #endif
>>>> -#define	eprints(a)	(void)write(STDERR_FILENO, (a), sizeof (a)-1)
>>>> -#define	eprintl(a)	(void)write(STDERR_FILENO, (a), strlen(a))
>>>> +#define	eprints(a)	do { int ret; ret = write(STDERR_FILENO, (a), sizeof (a)-1); } while (0)
>>>> +#define	eprintl(a)	do { int ret; ret = write(STDERR_FILENO, (a), strlen(a)); } while (0)
>>> What is the purpose of this change?
>> throw away the return value. newer gcc checks ignore (void).
> 
> That surprises me.  I always thought (void) was the correct way to tell
> the compiler we intentionally do not care about a return value.

seems like the checks do not like to be tricked easily with "(void)" :)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3623 bytes
Desc: S/MIME Cryptographic Signature
Url : http://lists.alioth.debian.org/pipermail/debburn-devel/attachments/20080226/eb83b394/attachment.bin