Date: Tue, 16 Nov 1999 19:26:28 +0100 From: Pierre Beyssac <beyssac@enst.fr> To: Kris Kennaway <kris@hub.freebsd.org> Cc: freebsd-current@FreeBSD.ORG Subject: Re: egcs -O breaks ping.c:in_cksum() Message-ID: <19991116192628.A83154@enst.fr> In-Reply-To: <Pine.BSF.4.10.9911151758400.16597-100000@hub.freebsd.org>; from Kris Kennaway on Mon, Nov 15, 1999 at 05:59:23PM -0800 References: <19991116002921.A39215@fasterix.frmug.org> <Pine.BSF.4.10.9911151758400.16597-100000@hub.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Nov 15, 1999 at 05:59:23PM -0800, Kris Kennaway wrote: > On Tue, 16 Nov 1999, Pierre Beyssac wrote: > > I've checked, the answer is no: apparently, in_cksum() in routed/rdisc.c > > is only called in two places, both with an even size. > > Can it hurt to pre-emptively fix it anyway in case some future change > pulls the rug out from underneath? We could, but since the danger is purely theoretical for now (and probably will stay that way forever), I don't see any advantage in cluttering up the code. Since routed is sometimes sync'ed from external sources, it would only make life harder for the people doing the merges. Plus, everyone steals in_cksum from ping, not from routed (at least, that's what I do :-) Since in_cksum is used in several places (there's another optimized copy in libstand), a cleaner solution would be to put it in some library. -- Pierre Beyssac pb@enst.fr To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?19991116192628.A83154>