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

Move command handling into queue #10

Closed
ProGTX opened this issue Jun 2, 2015 · 3 comments
Closed

Move command handling into queue #10

ProGTX opened this issue Jun 2, 2015 · 3 comments

Comments

@ProGTX
Copy link
Owner

ProGTX commented Jun 2, 2015

Get rid of the command_group class.
Already started with the introduction of the handler class: efeb82e

@ProGTX
Copy link
Owner Author

ProGTX commented Jun 27, 2015

Don't really need to get rid of the command_group class - for now I've moved it into namespace detail (374b6d6), did some trimming (09a3dec), and used it to provide a local compilation environment (f3a4e4f, not done yet).

But yes, in the end, the queue should be the only one to handle commands. But this can now be done by using local compilation environments (local command groups) and later joining them. This should allow for better optimizations, as all commands can be viewed at once.

@ProGTX
Copy link
Owner Author

ProGTX commented Oct 10, 2015

There were some problems with the local compilation environment, had to be removed in 10167ad.

@ProGTX
Copy link
Owner Author

ProGTX commented Oct 11, 2015

Basically done in 1ec0a7b, but needs #13. Closing.

@ProGTX ProGTX closed this as completed Oct 11, 2015
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