Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 16 Aug 2022 11:47:59 +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:  <YvuD7%2BoK6RZ/DLzH@FreeBSD.org>
In-Reply-To: <A9D9B817-8E18-49A5-87A0-BDD9FA748549@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>

next in thread | previous in thread | raw e-mail | index | archive | help
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.
Why can't it all be done in more coarse pieces, if not one commit?

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YvuD7%2BoK6RZ/DLzH>