Re: [PATCH 2/5] x86: Coding style fixes to arch/x86/kernel/bios_uv.c

From: H. Peter Anvin
Date: Mon Aug 04 2008 - 22:32:38 EST


Paolo Ciarrocchi wrote:

_much worse_? Wow, I thought the opposite.
Anyway, that's just a coding style patch, feel free to simply not apply it as
it's really a matter of personal taste but be sure that I'm not
blindly changing the code
accordingly to checkpatch. That might be almost true when I started
doing this kind of patches months
ago but it's definitely not true anymore.


In this case, the real problem is that the original code used a switch statement where it really should have used a table.

-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/