Carefully consider how const ExprPar& are used in ExprPredictor.cpp and INDExprPredictor.cpp, and how to fix that.

Question

Carefully consider how const ExprPar& are used in ExprPredictor.cpp and INDExprPredictor.cpp, and how to fix that.

Opened this issue 9 years ago · 1 comments

That way virtual function dispatch and such will work.

Answer 1 · 2015-04-03T18:41:54.000Z

Currently, the use of ExprPar& everywhere prevents proper dynamic dispatch and thus prevents the use of inheritance or design patterns to implement new versions of GEMSTAT. In a sense, this seemingly minor issue is the crux of many of our software engineering woes.