| To: | Adrian Bunk <bunk@xxxxxxxxx> |
|---|---|
| Subject: | Re: [2.6 patch] net/decnet/: misc possible cleanups |
| From: | Patrick Caulfield <patrick@xxxxxxxxxxxxxxx> |
| Date: | Wed, 15 Dec 2004 10:56:46 +0000 |
| Cc: | Steve Whitehouse <SteveW@xxxxxxx>, linux-decnet-user@xxxxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx |
| In-reply-to: | <20041214125838.GC23151@stusta.de> |
| Mail-followup-to: | Adrian Bunk <bunk@xxxxxxxxx>, Steve Whitehouse <SteveW@xxxxxxx>, linux-decnet-user@xxxxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx |
| References: | <20041214125838.GC23151@stusta.de> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.6+20040907i |
On Tue, Dec 14, 2004 at 01:58:38PM +0100, Adrian Bunk wrote: > The patch below contains the following possible cleanups: > - make needlessly global code static > - dn_fib.c: remove the write-only global variable dn_fib_info_cnt > - dn_fib.c: remove the unused global function dn_fib_rt_message > - dn_neigh.c: remove the unused global function dn_neigh_pointopoint_notify > - dn_timer.c: remove the fast timer code that isn't used > > Please review and comment on this patch. > Looks fine to me. I'm quite happy to lose the fast ack code - unused code is only a confusion to those reading it IMHO. If we do the delayed-ack code in future then it's easy enough to reinstate. Thanks. -- patrick |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: Re: [PATCH] e1000 poll behavior, jchapman |
|---|---|
| Next by Date: | Re: [2.6 patch] net/sched/: possible cleanups, Thomas Graf |
| Previous by Thread: | Re: [2.6 patch] net/decnet/: misc possible cleanups, Adrian Bunk |
| Next by Thread: | [2.6 patch] net/ethernet/eth.c: make a function static, Adrian Bunk |
| Indexes: | [Date] [Thread] [Top] [All Lists] |