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    [W:0.061 / U:29.836 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site