svn commit: r278828 - head/sys/netinet6

[Bjoern A. Zeeb]

> On 16 Feb 2015, at 01:30 , Gleb Smirnoff <glebius at FreeBSD.org> wrote:
> 
> On Mon, Feb 16, 2015 at 01:25:05AM +0000, Bjoern A. Zeeb wrote:
> B> 
> B> > On 16 Feb 2015, at 01:12 , Gleb Smirnoff <glebius at freebsd.org> wrote:
> B> > 
> B> > Author: glebius
> B> > Date: Mon Feb 16 01:12:20 2015
> B> > New Revision: 278828
> B> > URL: https://svnweb.freebsd.org/changeset/base/278828
> B> > 
> B> > Log:
> B> >  Factor out ip6_deletefraghdr() function, to be shared between IPv6
> B> >  stack and pf(4).
> B> > 
> B> >  Submitted by:	Kristof Provost
> B> >  Reviewed by:	ae
> B> >  Differential Revision:	D1764
> B> > 
> B> > Modified:
> B> >  head/sys/netinet6/frag6.c
> B> >  head/sys/netinet6/ip6_output.c
> B> 
> B> Why did it have to move file?

But the new function is in ip6_output and the consumer stays in frag6 and will be in pf?  So the new function moved to a different file without consumer rather than just on top of the old function?   Keep code logically together seems to make more sense to me.  What am I missing?