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

Provide more explicit system for command callbacks #96

Open
hawkerm opened this issue Jun 9, 2015 · 1 comment
Open

Provide more explicit system for command callbacks #96

hawkerm opened this issue Jun 9, 2015 · 1 comment

Comments

@hawkerm
Copy link
Member

hawkerm commented Jun 9, 2015

Should generalize the notion of commands returning information to the client.

Discussed making these returns asynchronous and putting more smarts in the client to manage messages, serialize delivery to the client code (to avoid threading concurrency issues), and generally manage the client architecture more robustly.

Radar should be reworked to fit this paradigm as a general available command.

@hawkerm
Copy link
Member Author

hawkerm commented Apr 22, 2016

@wortzmanb What are your thoughts on this? Should Death/Radar/Scan be separate interface methods which get called on the client semi-asynchronously from their main getNextCommand? This could remove the 'null'/state-management issue with radar, but also force implementing more of a 'memory'/coordination across the class.

@hawkerm hawkerm added the FUTURE label May 17, 2016
@hawkerm hawkerm modified the milestone: v1.2 - Hunger Baubles Jun 3, 2016
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