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

convert libs/protoboard to use BlackBoardInterfaceListener #186

Open
todo bot opened this issue Dec 4, 2019 · 1 comment
Open

convert libs/protoboard to use BlackBoardInterfaceListener #186

todo bot opened this issue Dec 4, 2019 · 1 comment
Assignees
Labels
enhancement Issue or PR describes or implements an enhancement pinned Issue will not be closed by stale bot todo 🗒️

Comments

@todo
Copy link

todo bot commented Dec 4, 2019

// TODO: This rather unspecific waker just triggers a loop(), so we need to go over all the interfaces
// each time and check where there's a message in the queue. This should be converted to a
// BlackBoardInterfaceListener, which calls a specific method and passes in the affected interface
// right away. That should allow significant simplification of all this template hackery.
waker_ = new fawkes::BlackBoardOnMessageWaker(blackboard, interface_, thread);
}


This issue was generated by todo based on a TODO comment in 98cfe88 when #171 was merged. cc @fawkesrobotics.
@todo todo bot added the todo 🗒️ label Dec 4, 2019
@vmatare vmatare changed the title This rather unspecific waker just triggers a loop(), so we need to go over all t... convert libs/protoboard to BlackBoardInterfaceListener Dec 4, 2019
@vmatare vmatare added the enhancement Issue or PR describes or implements an enhancement label Dec 4, 2019
@vmatare vmatare changed the title convert libs/protoboard to BlackBoardInterfaceListener convert libs/protoboard to use BlackBoardInterfaceListener Dec 4, 2019
@stale
Copy link

stale bot commented Jan 3, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix We won't fix this issue label Jan 3, 2020
@morxa morxa added pinned Issue will not be closed by stale bot and removed wontfix We won't fix this issue labels Jan 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue or PR describes or implements an enhancement pinned Issue will not be closed by stale bot todo 🗒️
Projects
None yet
Development

No branches or pull requests

2 participants