Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 16 Aug 2022 12:50:50 +0000
From:      Alexey Dokuchaev <danfe@freebsd.org>
To:        Dimitry Andric <dim@freebsd.org>
Cc:        Jessica Clarke <jrtc27@freebsd.org>, Konstantin Belousov <kostikbel@gmail.com>, src-committers <src-committers@freebsd.org>, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: 402dbdd98acc - main - Adjust function definition in arm's mv_common.c to avoid clang 15 warning
Message-ID:  <YvuSqgpQ/AnQKNVj@FreeBSD.org>
In-Reply-To: <4ed11043-d368-3084-bb21-2b794e9da748@FreeBSD.org>
References:  <202208151849.27FInHmh027652@gitrepo.freebsd.org> <Yvq1lQqmq3Mwv9ZJ@kib.kiev.ua> <F33312A7-B4B3-4D1A-9FE6-7CEF4E0F4BD9@freebsd.org> <A9D9B817-8E18-49A5-87A0-BDD9FA748549@FreeBSD.org> <YvuD7%2BoK6RZ/DLzH@FreeBSD.org> <4ed11043-d368-3084-bb21-2b794e9da748@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Aug 16, 2022 at 02:41:51PM +0200, Dimitry Andric wrote:
> On 2022-08-16 13:47, Alexey Dokuchaev wrote:
> > On Tue, Aug 16, 2022 at 12:10:04PM +0200, Dimitry Andric wrote:
> >> ...
> >> But I think it is better to have the definitions matching the
> >> declarations exactly. We should sweep through the whole tree and get
> >> rid of all K&R functions too. I believe Warner wanted to attempt that.
> > 
> > I won't comment on the technical side of things, but seeing this plethora
> > of identical commits is not just annoying, but pessimizes blaming as well.
> 
> I don't see any reason why this would "pessimize blaming"? What do you
> mean with that?

Sorry, I've meant bisecting.  Not sure why I used that word.

> Because then somebody else will probably complain "why everything in one
> commit, it should be fine-grained"?

I see; well, I believe a reasonable balance could be achieved, there are
a lot of numbers between 1 and 100. :)

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YvuSqgpQ/AnQKNVj>