Skip to content

unit tests for covariance #363

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
Merged
merged 1 commit into from
Sep 10, 2013
Merged

unit tests for covariance #363

merged 1 commit into from
Sep 10, 2013

Conversation

benjchristensen
Copy link
Member

  • refactoring so not everything for the entire Observable ends up in a single class

@benjchristensen
unit tests for covariance
6368578 
- refactoring so not everything for the entire Observable ends up in a single class
benjchristensen added a commit that referenced this pull request Sep 10, 2013
@benjchristensen
Merge pull request #363 from benjchristensen/reduce-covariance
e7ce270 
unit tests for covariance
@benjchristensen benjchristensen merged commit e7ce270 into ReactiveX:master Sep 10, 2013
@benjchristensen benjchristensen deleted the reduce-covariance branch September 10, 2013 01:47
@cloudbees-pull-request-builder
Copy link

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

rickbw pushed a commit to rickbw/RxJava that referenced this pull request Jan 9, 2014
@benjchristensen
Merge pull request ReactiveX#363 from benjchristensen/reduce-covariance
9e7c9cb 
unit tests for covariance
jihoonson pushed a commit to jihoonson/RxJava that referenced this pull request Mar 6, 2020
@RobWin
Return DecoratedCall from clone() to preserve its contract. (Reactive…
dd85ff6 
…X#363)

DecoratedCall should return an object of the same type (DecoratedCall)
to not break clone() contract on which other objects may relay on.
X.clone() cannot return Y, should return X.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@benjchristensen @cloudbees-pull-request-builder