Skip to content

[JTC] This set of cascading if-s should be refactored into something more readable. #326

Open
@bmagyar

Description

@bmagyar

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

No one assigned

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions