From: Jonathan Nieder Date: Mon, 05 Oct 2020 07:46:27 GMT Subject: Re: [PATCH 2/7] fsck_tree(): wrap some long lines Message-ID: <20201005074627.GE1166820@google.com> In-Reply-To: <20201005071935.GB2291074@coredump.intra.peff.net> Jeff King wrote: > Many calls to report() in fsck_tree() are kept on a single line and are > quite long. Most were pretty big to begin with, but have gotten even > longer over the years as we've added more parameters. Let's accept the > churn of wrapping them in order to conform to our usual line limits. > > Signed-off-by: Jeff King > --- > fsck.c | 48 ++++++++++++++++++++++++++++++++++++------------ > 1 file changed, 36 insertions(+), 12 deletions(-) "git show --word-diff" and "git show --check" show nothing obvious wrong (which in particular means there's no functional change); eyeing the patch, it looks good, too. Thanks. Reviewed-by: Jonathan Nieder