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

Implemented distinct and distinctUntilChanged variants using a comparator #380

Merged
merged 6 commits into from
Sep 13, 2013

Conversation

jmhofer
Copy link
Contributor

@jmhofer jmhofer commented Sep 13, 2013

As @abersnaze and @benjchristensen both agreed (see the #374 discussion) that they'd like the comparator versions of these two operations, too, I implemented them here.

@cloudbees-pull-request-builder

RxJava-pull-requests #277 FAILURE
Looks like there's a problem with this pull request

@jmhofer
Copy link
Contributor Author

jmhofer commented Sep 13, 2013

Hmm, today is the day of sporadic concurrency test failures, it seems. - This doesn't look like a problem caused by this PR.

@cloudbees-pull-request-builder

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

benjchristensen added a commit that referenced this pull request Sep 13, 2013
Implemented `distinct` and `distinctUntilChanged` variants using a comparator
@benjchristensen benjchristensen merged commit 141bb6b into ReactiveX:master Sep 13, 2013
@jmhofer jmhofer deleted the distinct-with-comparator branch September 14, 2013 09:12
rickbw pushed a commit to rickbw/RxJava that referenced this pull request Jan 9, 2014
Implemented `distinct` and `distinctUntilChanged` variants using a comparator
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.

None yet

3 participants