diff options
author | Vladimir Oltean <[email protected]> | 2024-10-23 16:52:47 +0300 |
---|---|---|
committer | Jakub Kicinski <[email protected]> | 2024-10-30 17:33:53 -0700 |
commit | a0af7162ccb501a22ac448ad94dad81757743725 (patch) | |
tree | 7373b5db23d6492eb61d772c8ee92c1f4ecda21e /tools/perf/scripts/python/sctop.py | |
parent | 2748697225c38a19666bba6a83afc6bf46ee16be (diff) |
net: dsa: clean up dsa_user_add_cls_matchall()
The body is a bit hard to read, hard to extend, and has duplicated
conditions.
Clean up the "if (many conditions) else if (many conditions, some
of them repeated)" pattern by:
- Moving the repeated conditions out
- Replacing the repeated tests for the same variable with a switch/case
- Moving the protocol check inside the dsa_user_add_cls_matchall_mirred()
function call.
This is pure refactoring, no logic has been changed, though some tests
were reordered. The order does not matter - they are independent things
to be tested for.
Signed-off-by: Vladimir Oltean <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/sctop.py')
0 files changed, 0 insertions, 0 deletions