[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <m2cyvm6jrw.fsf@gmail.com>
Date: Mon, 04 Dec 2023 15:59:31 +0000
From: Donald Hunter <donald.hunter@...il.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: netdev@...r.kernel.org, "David S. Miller" <davem@...emloft.net>, Eric
Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>, Jonathan
Corbet <corbet@....net>, linux-doc@...r.kernel.org, Jacob Keller
<jacob.e.keller@...el.com>, donald.hunter@...hat.com
Subject: Re: [PATCH net-next v1 3/6] tools/net/ynl: Add 'sub-message'
attribute decoding to ynl
Jakub Kicinski <kuba@...nel.org> writes:
> On Thu, 30 Nov 2023 21:49:55 +0000 Donald Hunter wrote:
>> @@ -510,7 +561,7 @@ class SpecFamily(SpecElement):
>> """
>> for op in self.yaml['operations']['list']:
>> if name == op['name']:
>> - return op
>> + return op
>> return None
>>
>> def resolve(self):
>
> Looks unrelated, plus the 'for' vs 'if' are still indented by 2.
Yep, I can drop this.
Powered by blists - more mailing lists