Messages in this thread |  | | | Date | Wed, 10 May 2006 05:00:57 -0400 (EDT) | | From | Steven Rostedt <> | | Subject | Re: [BUG] mtd redboot (also gcc 4.1 warning fix) |
| |
On Wed, 10 May 2006, Adrian Bunk wrote:
> On Tue, May 09, 2006 at 07:56:04PM -0700, Daniel Walker wrote: > > > unsigned long may not always be 32 bits, right ? This patch fixes the > > warning, but not the bug . > >... > > IOW, all your patch does is to hide a bug? > > Not exactly an improvement... >
Perfect example of what Andrew Morten stated in his keynote at LinuxTag. He mentioned patches that fixed warnings but not the problem that they warn about. He stated that these are very dangerous, because, not only is the problem not fixed, but now no one knows of the problem.
Daniel,
We appreciate the clean up of the kernel. Just try to focus on the obvious stuff, and maybe only flag those that you're not sure is still a problem. Email the author of the code, or kick his butt to get him to move and fix it.
Unintialized variables and bad compares may not be as trivial as they appear. Look closely at them to see if there isn't really a problem that lies underneath.
Thanks,
-- Steve
- 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/
|  |