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

Predicate.filter should return true if item must stay #285

Closed
IlyaGulya opened this issue Dec 9, 2016 · 5 comments
Closed

Predicate.filter should return true if item must stay #285

IlyaGulya opened this issue Dec 9, 2016 · 5 comments
Assignees

Comments

@IlyaGulya
Copy link

IlyaGulya commented Dec 9, 2016

Hi! I've just began using this awesome library, thanks for it!
But, i've found some strange thing: method Predicate.filter(Item, CharSequence) should return false, if item must stay in adapter.
This behaviour is opposite to same named methods in such default things, like RxJava, or Stream API and more others. I think it is not obvious for most of people. Shouldn't behaviour of adapter be the same as default in all other Java world?

@mikepenz
Copy link
Owner

@IlyaGulya thank you very much.

Hmmm I was referring to the methods name itself filter so true --> filter out. false don't filter out

But I see what you mean. I am not sure if it is such a good idea to change this behaviour at this point of the project

@IlyaGulya
Copy link
Author

@mikepenz I think you can add this to roadmap for next major release, for example

@FabianTerhorst
Copy link
Contributor

@IlyaGulya you can add suggestions to the #294 pr.

@IlyaGulya IlyaGulya mentioned this issue Dec 19, 2016
14 tasks
@FabianTerhorst
Copy link
Contributor

You can take a look into the latest commits from the pr. Filter should have the correct behavior now.

@IlyaGulya
Copy link
Author

Yeah, i've already seen them. Cool, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants