Skip to content

[FEATURE] Code cleanup. #267

@olilarkin

Description

@olilarkin

I would like it if we could significantly clean up the plug-in code after #242 is merged.

Please can we

  1. do something about the extremely badly named methods _GetNAM() and _GetIR() which actually "Stage" the new model/ir. It is very confusing to see methods named using the word Get, typically used for const Getter methods that do all sorts of sets!
  2. rename "NAM" in variable names to "Model"
  3. remove the use of this-> everywhere
  4. remove the underscore prefix from private methods
  5. use curly brackets for if/for loops even one-liners

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions