In message <f052b0f4-92e4-07c7-fec7-ada07fc23...@yandex.ru>, "Andrey V. 
Elsukov
" writes:
> This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
> --hploe8mi8b5CeIbHnELgS0brwST9uiIfp
> Content-Type: multipart/mixed; boundary="Ua6VdxS4jpKDm90X0cEvNMeMts7m7TLRO";
>  protected-headers="v1"
> From: "Andrey V. Elsukov" <bu7c...@yandex.ru>
> To: rgri...@freebsd.org
> Cc: src-committ...@freebsd.org, svn-src-all@freebsd.org,
>  svn-src-h...@freebsd.org
> Message-ID: <f052b0f4-92e4-07c7-fec7-ada07fc23...@yandex.ru>
> Subject: Re: svn commit: r327140 - head/sbin/ipfw
> References: <201712240711.vbo7bnrg045...@pdx.rh.cn85.dnsmgr.net>
> In-Reply-To: <201712240711.vbo7bnrg045...@pdx.rh.cn85.dnsmgr.net>
>
> --Ua6VdxS4jpKDm90X0cEvNMeMts7m7TLRO
> Content-Type: text/plain; charset=utf-8
> Content-Language: en-US
> Content-Transfer-Encoding: quoted-printable
>
> On 24.12.2017 10:11, Rodney W. Grimes wrote:
> >> -  uint32_t masks[2];
> >> -  int i;
> >> -  uint8_t cmd, rulenum;
> >>    ipfw_range_tlv rt;
> >>    char *msg;
> >>    size_t size;
> >> +  uint32_t masks[2];
> >> +  int i;
> >=20
> > Why do these 2 lines show up as null changes in the diff????
> > I can not see any change, not even white space.
>
> They are sorted by size according to style(9).

When I was still a grasshopper my sensei taught me not to batch commits. 
Looking back, this was wise advice as it is easier for those reading the 
logs to follow each intended change. The two commits should have been: 1) 
fix rule number truncation... and 2) sort size according to style(9).


-- 
Cheers,
Cy Schubert <cy.schub...@cschubert.com>
FreeBSD UNIX:  <c...@freebsd.org>   Web:  http://www.FreeBSD.org

        The need of the many outweighs the greed of the few.


_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to