Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

OperatorDematerialize#1083

Merged
benjchristensen merged 1 commit intoReactiveX:masterfrom
akarnokd:OperatorDematerialize
Apr 24, 2014
Merged

OperatorDematerialize#1083
benjchristensen merged 1 commit intoReactiveX:masterfrom
akarnokd:OperatorDematerialize

Conversation

@akarnokd
Copy link
Member

Operator Dematerialize

Issue #1060

Two notes:

  • Since the dematerialize() is an instance method, there is no correct type parametrization for the operator. Therefore, I just used a raw version in the call.
  • The original dematerialize could send out two onCompleted or an onError followed by an onCompleted because the materialize() always adds an onCompleted after the raw events. This affected operators due to unsafeSubscribe. This new version ensures this doesn't happen.

@cloudbees-pull-request-builder

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

@akarnokd akarnokd mentioned this pull request Apr 24, 2014
57 tasks
@benjchristensen benjchristensen merged commit 08085ac into ReactiveX:master Apr 24, 2014
@akarnokd akarnokd deleted the OperatorDematerialize branch April 24, 2014 21:23
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