Re: Some cleanup patches for: '...lvalues is deprecated'

From: Marcelo Tosatti
Date: Tue Jul 27 2004 - 08:58:11 EST


On Mon, Jul 05, 2004 at 01:59:21PM +0200, Joel Soete wrote:
> Hello Daniel,
>
> > > So just use
> > >
> > > buffer++;
> > >
> > > here, and the intent is then clear.
> >
> > Except C does not actually allow incrementing a void pointer, since
> > void does not have a size.
> That make better sense to me because aifair a void * was foreseen to pass
> any kind of type * as actual parameter?
> (So as far as I understand, the aritthm pointer sould be dynamic for the
> best 'natural' behaviour?)
>
> > You can't do arithmetic on one either. GNU
> > C allows this as an extension.
> >
> > It's actually this, IIRC:
> > buffer = ((char *) buffer) + 1;

Joel,

It seems the current code is working perfectly, generating correct
asm code.

Could you come up with a good enough reason to do this cleanup (as far as
I am concerned) in 2.4.x series?

Thanks
-
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/