Messages in this thread |  | | | Date | Tue, 27 Jul 2004 17:59:13 +0200 | | From | "Joel Soete" <> | | Subject | Re: Some cleanup patches for: '...lvalues is deprecated' |
| |
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@cyclades.com> > To: Joel Soete <soete.joel@tiscali.be> > Cc: Daniel Jacobowitz <dan@debian.org>, > Vojtech Pavlik <vojtech@suse.cz>, > Linux Kernel <linux-kernel@vger.kernel.org> > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |