Date: Mon, 26 Apr 2021 22:29:03 +0000
From: Rick Macklem <rmacklem@uoguelph.ca>
To: Neel Chauhan <nc@FreeBSD.org>, Kevin Bowling <kevin.bowling@kev009.com>
Cc: src-committers <src-committers@FreeBSD.org>, "dev-commits-src-all@FreeBSD.org" <dev-commits-src-all@FreeBSD.org>, "dev-commits-src-main@FreeBSD.org" <dev-commits-src-main@FreeBSD.org>
Subject: Re: git: e657f3de6dc2 - main - linuxkpi: Remove unneeded {} in atomic_dec_and_lock_irqsave()
Message-ID: <YTOPR0101MB09705574F7B6F13B4331D17CDD429@YTOPR0101MB0970.CANPRD01.PROD.OUTLOOK.COM>
In-Reply-To: <9b006e97a64d01d128d81bc09f0891ea@freebsd.org>
References: <202104261525.13QFPrbg086311@gitrepo.freebsd.org> <CAK7dMtDuJ_4z2Xwej1hpGcXC6LaP5Wwo5ymbA_4-tds22spOgg@mail.gmail.com>, <9b006e97a64d01d128d81bc09f0891ea@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Neel wrote:=0A=
>On 2021-04-26 09:47, Kevin Bowling wrote:=0A=
>> I'm not sure all the context or conversation here but the convention=0A=
>> is to not use bare return values, i.e in style(9) "Values in return=0A=
>> statements should be enclosed in parentheses." and that's what was=0A=
>> asked to be changed on this mailing list.=0A=
Just fyi to everyone, there is this in style(9):=0A=
In general code can be considered =93new code=94 when it makes up abou=
t 50%=0A=
or more of the file(s) involved. This is enough to break precedents i=
n=0A=
the existing code and use the current style guidelines.=0A=
=0A=
As such, if the "return 0;" predates this patch series, Neel is correct=0A=
to use "return 1;", since that precedent has already been established.=0A=
I'll admit I see the above ignored a lot and personally don't care if=0A=
the above generality is followed, but it is in style(9) and I do=0A=
think a consistent style is preferred over a jumble within a source file.=
=0A=
=0A=
rick=0A=
=0A=
The review: https://reviews.freebsd.org/D29988=0A=
=0A=
I believe I was asked to do this in the review.=0A=
=0A=
-Neel=0A=
>=0A=
> Can you use and link to Phabricator for your src commits? As much as=0A=
> possible it is preferable to get it right in one go, for MFCs,=0A=
> bisection, etc and this kind of churn should be preventable with quick=0A=
> reviews. Feel free to tag me as a reviewer.=0A=
=0A=
Sure, will do next time.=0A=
=0A=
=0A=
> Regards,=0A=
> Kevin=0A=
=0A=
-Neel=0A=
=0A=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YTOPR0101MB09705574F7B6F13B4331D17CDD429>