Skip to content

Conversation

@forki
Copy link
Contributor

@forki forki commented Sep 5, 2016

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point. done

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mhm after thinking about it. All the other listset functions have the first parameter f which is the projection. In this sense it makes sense to revert my last commit and keep it named contains.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The real question is: can we use a different data structure!?

It needs to be a set that allows to pass in own equality semantics.

@KevinRansom
Copy link
Contributor

Looks good to me,

thanks for this contribution

Kevin

@KevinRansom KevinRansom reopened this Sep 11, 2016
@KevinRansom KevinRansom merged commit e7d09b2 into dotnet:master Sep 11, 2016
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