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

Virtual ofParameter reInit() #7900

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

artificiel
Copy link
Contributor

Following the isInit() implementation, it makes sense to push reInit() up into the ofAbstractParameter so it can also be done without unwrapping the parameter type. It can then also be applied on ofParameterGroup (where it reInits all the children).

Added verbose logs to notify users that calling reInit() on the special-case ofParameter<void> has no effect. (Also inverted the result of ofParameter<void>::isInit() (from false to true) as the general use case of iterating a group will probably be to look for non-init values, so the <void> types have more chances of being unobtrusive returning true).

@artificiel artificiel changed the title Virtualreinit Virtual ofParameter reInit() Feb 13, 2024
@artificiel
Copy link
Contributor Author

while we're in the subject, I guess it's an ofSite question but how does the online docs get re-generated? as of now the reference is 6-8 years old. (NB I've previously tried to install the toolchain but got stuck).

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

Successfully merging this pull request may close these issues.

None yet

2 participants