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

[meta] Cut out dispatchers #11

Open
1 of 5 tasks
Hywan opened this issue Jan 9, 2015 · 1 comment
Open
1 of 5 tasks

[meta] Cut out dispatchers #11

Hywan opened this issue Jan 9, 2015 · 1 comment

Comments

@Hywan
Copy link
Member

Hywan commented Jan 9, 2015

The Basic dispatcher is able to dispatch a request/task on several callables: function, closure, class+method, object+method…

The idea is to extract each one of these forms as a standalone dispatcher. Then, when they will be tested, we could introduce an Aggregate dispatcher. Finally, the Basic dispatcher would be a special usage of the Aggregate dispatcher that aggregates all the existing dispatchers.

The roadmap is:

  • ensuring better performances for all dispatchers,
  • ensuring better testability for all dispatchers,
  • do not introduce BC break (not a lot at least, one is necessary, see incoming issues or PR),
  • ensuring a better modularity by choosing/aggregating dispatchers.

Progress:

  • Function (incl. Closure) dispatcher,
  • ClassMethod dispatcher,
  • ObjectMethod dispatcher,
  • Aggregate dispatcher,
  • Basic dispatcher.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

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

No branches or pull requests

2 participants