lkml.org 
[lkml]   [2004]   [Jul]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 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/

\
 
 \ /
  Last update: 2005-03-22 14:04    [from the cache]
©2003-2011 Jasper Spaans