Re: CVS commit: src/lib/libc/inet

[Brian Ginsbach <ginsbach%NetBSD.org@localhost>]

On Sat, Jan 19, 2008 at 10:43:45AM -0500, Christos Zoulas wrote:
> On Jan 19,  3:47am, ginsbach%NetBSD.org@localhost (Brian Ginsbach) wrote:
> -- Subject: Re: CVS commit: src/lib/libc/inet
> 
> | I don't think this was necessary as itojun FIXED this problem 6+ years
> | ago pre-source move (src/lib/libc/net/inet_network.c 1.14) in
> | NetBSD.  If NetBSD is going to go with the "standard" solution then
> | we should remove the lines added by itojun (lines 105-106 rev 1.2).
> 
> Yes, but it does no harm to move the check outside the if statement and
> it is better to match the bind source as closely as possible. Remember
> that there is another copy of that file in dist.
> 

Never said it did.  If the goal is to keep it looking as close to
the bind source then we really should remove itojun's change from
the libc version.  It is now redundant code.

Note I had no problem with changing the bind version in dist.