Messages in this thread | | | Date | Wed, 26 Sep 2007 23:08:19 +0200 | From | roel <> | Subject | Re: [PATCH 2/4] dmapool: Validate parameters to dma_pool_create |
| |
Matthew Wilcox wrote: > On Wed, Sep 26, 2007 at 09:47:41PM +0200, roel wrote: >> The brackets in the first if/else are not required, and you could combine the two statements: > > You mean braces, not brackets. And I find this little fetish of yours > highly disturbing. I prefer to use braces, and will continue to do so, > regardless of your nitpicking.
Well as you say it, you like the braces, so it appears to be your fetish. Of course you don't have to make any changes, I am just reporting them cause they aren't needed. No need for the offensive tone either.
Roel - 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/
| |