Date: Mon, 16 Feb 2015 08:38:04 +0000 From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: Gleb Smirnoff <glebius@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r278828 - head/sys/netinet6 Message-ID: <114163CC-5599-4A12-92B6-5B106EF86263@FreeBSD.org> In-Reply-To: <20150216033656.GD15484@FreeBSD.org> References: <201502160112.t1G1CKd0074570@svn.freebsd.org> <C993C12A-0F69-4BBF-86E8-D361B3F3A764@FreeBSD.org> <20150216013000.GB15484@FreeBSD.org> <8750BD23-1384-4065-8368-C332F6683814@FreeBSD.org> <20150216033656.GD15484@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On 16 Feb 2015, at 03:36 , Gleb Smirnoff <glebius@freebsd.org> wrote: >=20 > On Mon, Feb 16, 2015 at 01:34:09AM +0000, Bjoern A. Zeeb wrote: > B>=20 > B> > On 16 Feb 2015, at 01:30 , Gleb Smirnoff <glebius@FreeBSD.org> = wrote: > B> >=20 > B> > On Mon, Feb 16, 2015 at 01:25:05AM +0000, Bjoern A. Zeeb wrote: > B> > B>=20 > B> > B> > On 16 Feb 2015, at 01:12 , Gleb Smirnoff = <glebius@freebsd.org> wrote: > B> > B> >=20 > B> > B> > Author: glebius > B> > B> > Date: Mon Feb 16 01:12:20 2015 > B> > B> > New Revision: 278828 > B> > B> > URL: https://svnweb.freebsd.org/changeset/base/278828 > B> > B> >=20 > B> > B> > Log: > B> > B> > Factor out ip6_deletefraghdr() function, to be shared = between IPv6 > B> > B> > stack and pf(4). > B> > B> >=20 > B> > B> > Submitted by: Kristof Provost > B> > B> > Reviewed by: ae > B> > B> > Differential Revision: D1764 > B> > B> >=20 > B> > B> > Modified: > B> > B> > head/sys/netinet6/frag6.c > B> > B> > head/sys/netinet6/ip6_output.c > B> > B>=20 > B> > B> Why did it have to move file? > B>=20 > B> 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? >=20 > You are right. I somehow misread it as function being in frag6.c and = consumer > in ip6_output.c. Will fix. Thanks!
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?114163CC-5599-4A12-92B6-5B106EF86263>