Open
Description
This set of cascading if
-s should be refactored into something more readable.
I suggest 2 things:
- make the selection of choosing the command somehow more explicit with different variable names, etc:
tmp_command_
is not very intuitive - refactor all the
assign_interface_from_point
uses into a function that gets a parameter, the command (whether it's desired state or the pid command)
Originally posted by @bmagyar in #225 (comment)
Metadata
Metadata
Assignees
Type
Projects
Status
No status