Re: [PATCH -mm] sys_semctl gcc 4.1 warning fix

Daniel Walker
2006-05-10T15:38:44Z

On Wed, 2006-05-10 at 16:39 +0100, Alan Cox wrote:
> On Mer, 2006-05-10 at 08:06 -0700, Daniel Walker wrote:
> > > Because while the warning is present people will check it now and again.
> >
> > But it's pointless to review it, in this case and for this warning .
>
> Then why did you review it ?

So I wouldn't have to see that warning again ..

> It reminds people that it does need checking, and ensures now and then
> people do check that there isn't actually a bug there.

I don't see a reason why it needs checking .. People are just going to
spin their wheel reviewing code that's been reviewed .. Maybe someone
like me will write a patch to fix this warning , and we'll see this
process happening all over again ..

Daniel