On Mon, 28 May 2007 03:11:04 +0000 "young dave" <[EMAIL PROTECTED]> wrote:
> Hi, > > And then there's the supercompact form. > > > > while (len--) { > > hash = partial_name_hash(tolower(*name++), hash); > > } > > > > But I do not like the last one at all. The first one is the best, because > > it clearly separates the condition and iteration parts of the expression, > > while STILL being only three lines long. Or two, if you omit the braces. > > (But you shouldn't.) > > > > IMO, I like the last one, but I prefer to keep the original author's > one, I only remove the unnecessary tolower function. > What do you think about this , Andrew? > Don't care much. The code as it stands is suitably paranoid about buggy implementations of tolower() which evaluate their arg more than once ;) Your email client replaces tabs with spaces. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/