Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

simplify class CParam<Param_t> #7

Open
qPCR4vir opened this issue Oct 8, 2018 · 0 comments
Open

simplify class CParam<Param_t> #7

qPCR4vir opened this issue Oct 8, 2018 · 0 comments

Comments

@qPCR4vir
Copy link
Owner

qPCR4vir commented Oct 8, 2018

Currently it can accept a reference to the used simple program parameter. But this complicate this clase, causing code duplication and introducing many nuances for it use: with the get() or directly, or even creating dangling references. It also have an internal value, "just in case" no external reference is passed. We better make this case the only available, so this will be the only place where the parameter is. It also introduce possible value duplication in base classes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant