Messages in this thread | | | Date | Wed, 14 Nov 2001 03:15:49 -0500 | From | Jeff Garzik <> | Subject | Re: [PATCH] reformat mtrr.c to conform to CodingStyle |
| |
David Weinehall wrote: > I don't think Lindent does everything 100% correct; at least its > formatting of switch/case does look a little fishy: > > switch (option) { > case 1: > /* blaha */ > > That feels kind of odd compared to the rest of the codingstyle. > > Comments?!
hehe now we really get into personal preference.
I like the above case statement style, because I see no value in additional indentation. The code is presented one level deeper than the switch statement, which is key. 'case' keywords merely separate the properly-indented code. IMHO of course :)
-- Jeff Garzik | Only so many songs can be sung Building 1024 | with two lips, two lungs, and one tongue. MandrakeSoft | - nomeansno
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |