[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240219121600.03db3457@kernel.org>
Date: Mon, 19 Feb 2024 12:16:00 -0800
From: Jakub Kicinski <kuba@...nel.org>
To: Stephen Hemminger <stephen@...workplumber.org>
Cc: Breno Leitao <leitao@...ian.org>, davem@...emloft.net,
pabeni@...hat.com, edumazet@...gle.com, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, horms@...nel.org, Johannes Berg
<johannes.berg@...el.com>
Subject: Re: [PATCH net-next v2] net: sysfs: Do not create sysfs for non BQL
device
On Fri, 16 Feb 2024 09:29:05 -0800 Stephen Hemminger wrote:
> Various compilers will warn about missing parens in that expression.
> It is valid but mixing & and || can be bug trap.
$ git grep ' & [A-Z0-9_]* ||' | wc -l
855
Let's not sprinkle parenthesis in correct code because some old
compiler somewhere may doubt our ability to remember the precedence
of 14 operators.
Powered by blists - more mailing lists