Re: [PATCH] usb: musb_core: mark expected switch fall-through

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 30, 2017 at 09:51:18AM +0100, Greg Kroah-Hartman wrote:
> On Fri, Oct 27, 2017 at 11:56:14AM -0500, Bin Liu wrote:
> > On Fri, Oct 27, 2017 at 06:49:49PM +0200, Greg Kroah-Hartman wrote:
> > > On Fri, Oct 27, 2017 at 11:44:47AM -0500, Bin Liu wrote:
> > > > Hi,
> > > > 
> > > > On Mon, Oct 23, 2017 at 10:10:43PM -0500, Gustavo A. R. Silva wrote:
> > > > > In preparation to enabling -Wimplicit-fallthrough, mark switch cases
> > > > > where we are expecting to fall through.
> > > > > 
> > > > > Addresses-Coverity-ID: 1397608
> > > > > Signed-off-by: Gustavo A. R. Silva <garsilva@xxxxxxxxxxxxxx>
> > > > 
> > > > Applied with the above Coverity-ID message removed.
> > > 
> > > Why?  It's good to track these things.
> > 
> > hmm, I thought we don't want non-kernel related information in the commit
> > message.
> 
> This is kernel-related information.
> 
> We don't want foolish "Id:" lines, from gerrit instances, as they are no
> help at all because we can not do anything with them.
> 
> We _do_ want identifiers that are globally unique that provide real
> information, like public bugzilla ids, and tools like coverity.  So for
> things like this, always keep them on.

Thanks for clarifying this.

> 
> I've added it back by hand now.

Thanks,
-Bin.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux