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.

Sign up for GitHub

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

Jump to bottom

EventProcessor refactoring #21488

Merged
merged 2 commits into from Nov 30, 2017
Merged

EventProcessor refactoring #21488

merged 2 commits into from Nov 30, 2017

Commits on Nov 29, 2017

  1. SerialTaskQueueChain now works with mutable lambdas 

    mutable lambdas have non-const operator() which prohibits passing iAction as a const reference.
    @Dr15Jones
    Dr15Jones committed Nov 29, 2017
    Copy the full SHA
    4538287 View commit details
    Browse the repository at this point in the history

  2. Use WaitingTaskHolder instead of bare pointer to task 

    Changed the function handleNextEventForStreamAsync to take as argument a WaitingTaskHolder instead of a bare pointer. This simplifies the ownership logic.
In addition, moved the construction of 'recursionTask' to be within the lambda which reads from the source since that is the only place it is needed.
    @Dr15Jones
    Dr15Jones committed Nov 29, 2017
    Copy the full SHA
    c715646 View commit details
    Browse the repository at this point in the history