GitLab

allow the use of previously assigned variables in epilogue block in later equations in epilogue block

Introducing a new DataTree::getVariable() const method was necessary in the process.

In particular, it is necessary to turn back DataTree::AddVariable() into a
non-virtual method, because it is called from DataTree's constructor. Enforcing
the absence of leads/lags is now done using a new boolean DataTree::is_static.

Take advantage of the new copy constructor for handling
PlannerObjectiveStatement more elegantly.

Unfortunately it is not possible to implement *move* constructor / assigment
operators, because the reference ExprNode::datatree is not mutable.

The syntax makes clearer the distinction between a function call and an
initialization. Also, it prevents bugs due to implicit type conversions.

Incidentally, replace the IS_* macros by inline functions.

This code is broken since the temporary terms array changes. It is probably not
worth maintaining it.

- M and MEX files are now under +${MODELNAME}/
- bytecode and C source now under ${MODELNAME}/model/

In C++11, it is no longer necessary to put a space between two consecutive ">".

Performed using modernize-use-emplace from clang-tidy.

Manual intervention was needed to transform a NULL into nullptr in SymbolTable.cc.

https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-emplace.html

Performed using modernize-use-auto from clang-tidy.

https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-auto.html

Performed using modernize-loop-convert from clang-tidy.

Manual intervention was needed in MacroValue.cc because of a variable name
capture issue.

https://clang.llvm.org/extra/clang-tidy/checks/modernize-loop-convert.html

Consequently, change various function calls where tef_terms was empty, passing
an (empty) rvalue instead of an lvalue.