Date: Thu, 19 Jul 2018 16:48:18 -0400 From: Michael Tuexen <tuexen@freebsd.org> To: Alexey Dokuchaev <danfe@FreeBSD.org> Cc: cem@freebsd.org, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers <src-committers@freebsd.org> Subject: Re: svn commit: r336503 - in head/sys: netinet netinet6 Message-ID: <4E76E1DC-DB70-47D1-A59D-E99058C00D53@freebsd.org> In-Reply-To: <20180719202738.GA69018@FreeBSD.org> References: <201807191933.w6JJXhof018383@repo.freebsd.org> <CAG6CVpWMWAZW_K47X1kXFS-JJo8Nc3zewKCzjRb7tr5BUsRAUA@mail.gmail.com> <F9ECD259-049F-45FB-A104-678749564BCC@freebsd.org> <20180719202738.GA69018@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On 19. Jul 2018, at 16:27, Alexey Dokuchaev <danfe@FreeBSD.org> wrote: > > On Thu, Jul 19, 2018 at 04:23:27PM -0400, Michael Tuexen wrote: >>> On 19. Jul 2018, at 16:11, Conrad Meyer <cem@freebsd.org> wrote: >>> These changes mostly seem like style regressions, rather than >>> improvements. (Not to mention, largescale churn like this makes it >>> more difficult for future readers to research history (confusing tools >>> like "git/svn blame").) >> >> We have upstream code and run this to an formatter tool which depends >> on ident, which has been changed recently. Unfortunately, I also ran >> an older version of the script, so I reverted this in >> https://svnweb.freebsd.org/changeset/base/336508 >> and committed a version really only showing the differences related to >> the ident changes in >> https://svnweb.freebsd.org/changeset/base/336511 > > Thanks for fixing, but you might find doing "svn diff" prior to commit > very handy next time (no offense). I did actually making sure the there are only whitespace changes. However, I just take what the tool produces (for years). Only after doing svn diff for a functional change, where no whitespace changes should be, but actually were, I realised my usage error and reverted... Best regards Michael > > ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4E76E1DC-DB70-47D1-A59D-E99058C00D53>