Re: [Q] warning BUG() related fixing and janitors question

[Dan Carpenter <dan.carpenter@oracle.com>]

On Mon, Apr 16, 2012 at 06:12:07PM -0300, Ezequiel García wrote:
> Hi Dan,
> 
> 2012/4/16 Dan Carpenter <dan.carpenter@oracle.com>:
> >
> > Don't start messing in scheduler code if you have any doubts about
> > what you are doing.  My advise is don't send a patch for this
> > warning.  Most times people leave BUG() enabled so we don't really
> > care about the warning.
> 
> Okey. Perhaps I picked the wrong example, there are a whole bunch of warnings
> related to the same BUG() stuff, here's a template example:
> 
> void *p;
> 
> switch (something) {
>   case a:
>     p = foo();
>   case b:
>     p = bar();
>   default:
>     BUG();
> }
> 
> dosomething(p);
> 
> --
> 
> This will trigger warning "p could be used uninitialized" or something.
> I know most of us just leave BUG on, so you say these warnings aren't important?

Yeah.  Probably best to ignore them.

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html