On Tue, Jun 11, 2013 at 06:21:08PM -0400, Vince Weaver wrote: > On Tue, 11 Jun 2013, Dave Jones wrote: > > > yeah, that's what I've done in a lot of other places (actually BUG() in those cases). > > For the most part they're present just to shut up -Wswitch-default, which > > has caught a few cases in the past where I've missed an option. > > I'll go back through and check out all of the switch statements (there > are a lot of them) and make sure they handle the default case > consistently. > > > ok, I'll apply what you sent so far. (and hold off on running it through Lindent for now, > > you really don't like whitespace huh? ;) > > Not enough whitespace? Or over-use of tabs? Mostly stuff like this. .. if((foo&bar)==0) i=rand(); instead of if ((foo & bar) == 0) i = rand(); indent -kr gets it mostly right. I tried bending the kernels scripts/Lindent to fit, but I'm still not 100% happy. It's good for a start point though. Dave -- To unsubscribe from this list: send the line "unsubscribe trinity" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
