Skip to content

Notifications for the allocation averse.#970

Merged
benjchristensen merged 1 commit intoReactiveX:masterfrom
abersnaze:notification-lite
Mar 20, 2014
Merged

Notifications for the allocation averse.#970
benjchristensen merged 1 commit intoReactiveX:masterfrom
abersnaze:notification-lite

Conversation

@abersnaze
Copy link
Contributor

I've seen and used the sentinel pattern in a couple of places in the implementation of operators. I figured I'd formalize the hacks into one place where bugs can coalesce. As a bonus the code that uses it looks a bit cleaner now.

@cloudbees-pull-request-builder

RxJava-pull-requests #911 SUCCESS
This pull request looks good

@benjchristensen
Copy link
Member

Nice abstraction.

benjchristensen added a commit that referenced this pull request Mar 20, 2014
Notifications for the allocation adverse.
@benjchristensen benjchristensen merged commit 5ca609b into ReactiveX:master Mar 20, 2014
@benjchristensen benjchristensen deleted the notification-lite branch March 20, 2014 16:25
@abersnaze abersnaze changed the title Notifications for the allocation adverse. Notifications for the allocation averse. Mar 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants