netdev
[Top] [All Lists]

Re: [2.6 patch] net/decnet/: misc possible cleanups

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>