netdev
[Top] [All Lists]

Re: [RTNETLINK] Convert RTM_* to enum

To: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [RTNETLINK] Convert RTM_* to enum
From: "David S. Miller" <davem@xxxxxxxxxxxxx>
Date: Tue, 14 Sep 2004 21:38:37 -0700
Cc: jmorris@xxxxxxxxxx, yoshfuji@xxxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <20040915034748.GA952@gondor.apana.org.au>
References: <20040915020942.GA32721@gondor.apana.org.au> <Xine.LNX.4.44.0409142337130.28280-100000@thoron.boston.redhat.com> <20040915034748.GA952@gondor.apana.org.au>
Sender: netdev-bounce@xxxxxxxxxxx
On Wed, 15 Sep 2004 13:47:48 +1000
Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> wrote:

> On Tue, Sep 14, 2004 at 11:38:21PM -0400, James Morris wrote:
> > 
> > Having the enums as well as the defines is messy, I wonder if it's really
> > worth it.
> 
> I think the enum is definitely worth it for reducing the churn on
> the MAX value.
> 
> I personally don't see a point to the defines since the user-space
> appliations should not change behaviour based on compile-time
> settings.  However, others seem to have a different opinion on that.

Right.  If we start using defines we have to keep them around.
I know it's bogus for people to ifdef this stuff, but we know
they do, and it's in bad taste to knowingly break stuff like that.

Anyways, I'll apply your patch Herbert, thanks.

<Prev in Thread] Current Thread [Next in Thread>