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

From: Joel Soete
Date: Tue Jul 27 2004 - 11:02:25 EST


Marcelo,

Thanks first for your attention.
Sorry also for delaying this works but I was a bit busy elsewhere.

> -- Original Message --
> Date: Tue, 27 Jul 2004 09:54:32 -0300
> From: Marcelo Tosatti <marcelo.tosatti@xxxxxxxxxxxx>
> To: Joel Soete <soete.joel@xxxxxxxxxx>
> Cc: Daniel Jacobowitz <dan@xxxxxxxxxx>,
> Vojtech Pavlik <vojtech@xxxxxxx>,
> Linux Kernel <linux-kernel@xxxxxxxxxxxxxxx>
> Subject: Re: Some cleanup patches for: '...lvalues is deprecated'
>
>
> 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?
>
My first attention was to cleanup some warning of type "use of cast expression
as lvalue is deprecated"
with gcc-3.3.4. But afaik, right now, there are just few warning which didn't
break the asm code.

I will try to come back asap with a better solution (at least I hope ;)
)

Thanks again,
Joel

---------------------------------------------------------------------------
Tiscali ADSL LIGHT, 19,95 EUR/mois pendant 6 mois, c'est le moment de faire
le pas!
http://reg.tiscali.be/default.asp?lg=fr




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