| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [RTNETLINK] Convert RTM_* to enum, Herbert Xu |
|---|---|
| Next by Date: | Re: [RTNETLINK] Convert RTM_* to enum, Herbert Xu |
| Previous by Thread: | Re: [RTNETLINK] Convert RTM_* to enum, Herbert Xu |
| Next by Thread: | Re: [RTNETLINK] Convert RTM_* to enum, Herbert Xu |
| Indexes: | [Date] [Thread] [Top] [All Lists] |