On Fri, Mar 13, 2020 at 12:23:26PM +0100, Tim Düsterhus wrote:
> Willy,
> 
> Am 09.03.20 um 17:05 schrieb Tim Duesterhus:
> > gcc complains (non-rightfully):
> > 
> >> include/common/buf.h: In function 'br_head_pick':
> >> include/common/debug.h:62:4: warning: ignoring return value of 'write', 
> >> declared with attribute warn_unused_result [-Wunused-result]
> >>     (void)write(2, msg, strlen(msg));                      \
> >>     ^
> >> include/common/debug.h:57:35: note: in expansion of macro '__BUG_ON'
> >>  #define _BUG_ON(cond, file, line) __BUG_ON(cond, file, line)
> >>                                    ^
> >> include/common/debug.h:56:22: note: in expansion of macro '_BUG_ON'
> >>  #define BUG_ON(cond) _BUG_ON(cond, __FILE__, __LINE__)
> >>                       ^
> >> include/common/buf.h:1011:2: note: in expansion of macro 'BUG_ON'
> >>   BUG_ON(r->area != BUF_RING.area);
> >>   ^
> 
> While checking my list of outgoing patches I noticed that this one
> wasn't acknowledged yet. It will become important with:
> https://github.com/haproxy/haproxy/issues/546

Indeed, just found it in my queue. However we usually use it
differently, with the function instead of the variable. Do you
mind if I adapt it ?

Willy

Reply via email to