-
Notifications
You must be signed in to change notification settings - Fork 192
Closed
Labels
enhancementNew feature or requestNew feature or requestpriority:lowLow priority issuesLow priority issues
Description
I would like it if we could significantly clean up the plug-in code after #242 is merged.
Please can we
- 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!
- rename "NAM" in variable names to "Model"
- remove the use of this-> everywhere
- remove the underscore prefix from private methods
- use curly brackets for if/for loops even one-liners
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestpriority:lowLow priority issuesLow priority issues