diff options
| author | Matthieu Baerts (NGI0) <matttbe@kernel.org> | 2025-09-09 23:07:54 +0200 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2025-09-10 18:09:00 -0700 |
| commit | f6259ba70e7e056ae626b393c908500a5d6e8755 (patch) | |
| tree | 7acb62c7c973c217ecf8943ad43d27f7f8c29af8 /tools/net/ynl/pyynl/ynl_gen_c.py | |
| parent | tools: ynl: use 'cond is None' (diff) | |
| download | linux-f6259ba70e7e056ae626b393c908500a5d6e8755.tar.gz linux-f6259ba70e7e056ae626b393c908500a5d6e8755.zip | |
tools: ynl: check for membership with 'not in'
It is better to use 'not in' instead of 'not {element} in {collection}'
according to Ruff.
This is linked to Ruff error E713 [1]:
Testing membership with {element} not in {collection} is more readable.
Link: https://docs.astral.sh/ruff/rules/not-in-test/ [1]
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Reviewed-by: Donald Hunter <donald.hunter@gmail.com>
Reviewed-by: Asbjørn Sloth Tønnesen <ast@fiberby.net>
Link: https://patch.msgid.link/20250909-net-next-ynl-ruff-v1-8-238c2bccdd99@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/net/ynl/pyynl/ynl_gen_c.py')
| -rwxr-xr-x | tools/net/ynl/pyynl/ynl_gen_c.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/net/ynl/pyynl/ynl_gen_c.py b/tools/net/ynl/pyynl/ynl_gen_c.py index c7fb8abfd65e..101d8ba9626f 100755 --- a/tools/net/ynl/pyynl/ynl_gen_c.py +++ b/tools/net/ynl/pyynl/ynl_gen_c.py @@ -638,7 +638,7 @@ class TypeBitfield32(Type): return '.type = YNL_PT_BITFIELD32, ' def _attr_policy(self, policy): - if not 'enum' in self.attr: + if 'enum' not in self.attr: raise Exception('Enum required for bitfield32 attr') enum = self.family.consts[self.attr['enum']] mask = enum.get_mask(as_flags=True) |