Date: Thu, 29 Mar 2007 13:52:12 +1200 From: Andrew Thompson <thompsa@freebsd.org> To: Greg 'groggy' Lehey <grog@FreeBSD.org> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Maksim Yevmenkin <emax@FreeBSD.org>, cvs-all@FreeBSD.org Subject: Re: Giving in to Coverity (was: cvs commit: src/sys/netgraph/bluetooth/l2cap ng_l2cap_cmds.c) Message-ID: <20070329015212.GA97061@heff.fud.org.nz> In-Reply-To: <20070329012834.GC79742@wantadilla.lemis.com> References: <200703282125.l2SLPuR9058727@repoman.freebsd.org> <20070329012834.GC79742@wantadilla.lemis.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Mar 29, 2007 at 10:58:34AM +0930, Greg 'groggy' Lehey wrote: > On Wednesday, 28 March 2007 at 21:25:56 +0000, Maksim Yevmenkin wrote: > > emax 2007-03-28 21:25:56 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/netgraph/bluetooth/l2cap ng_l2cap_cmds.c > > Log: > > Try to silence Coverity by adding (void) in front of function call. > > Also add a comment, explaining why return value is not being checked. > > I hope Coverity isn't going to force us to add unnecessary casts to > function calls. Well no, you can always silence Coverity by just marking it as a false bug. Andrew
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070329015212.GA97061>