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

Proposed callback manager. #1889

Closed
wants to merge 2 commits into from
Closed

Proposed callback manager. #1889

wants to merge 2 commits into from

Conversation

Zeno-
Copy link
Contributor

@Zeno- Zeno- commented Nov 30, 2014

Currently not used but can in future be utilized for fast 'caching' of settings and any other use where callback functions are useful

it != m_callbacks.end(); ++it) {
if (*it == function)
return true;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#include <algorithm>
...
return std::find(m_callbacks.begin(), m_callbacks.end(), function) != m_callbacks.end();

@ShadowNinja ShadowNinja added @ Startup / Config / Util Maintenance Tasks to keep the codebase and related parts in order, including architectural improvements labels Dec 1, 2014
Currently not used but can in future be utilized for fast 'caching' of settings and any other use where callback functions are useful
@Zeno- Zeno- added the WIP The PR is still being worked on by its author and not ready yet. label Dec 1, 2014
@Zeno- Zeno- closed this Dec 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Tasks to keep the codebase and related parts in order, including architectural improvements @ Startup / Config / Util WIP The PR is still being worked on by its author and not ready yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants