Skip to content

Implement PartialEq, Eq, PartialOrd, Ord on SmallVec. #22

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

Merged
merged 1 commit into from
May 24, 2016

Conversation

cfallin
Copy link
Contributor

@cfallin cfallin commented May 24, 2016

Currently one can compare slices, but having the traits on the SmallVec itself is slightly more convenient. Perhaps there's a reason these traits aren't yet implemented, but if not, here's a PR!


This change is Reviewable

@SimonSapin
Copy link
Member

Looks good, thanks!

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 946aab6 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

⌛ Testing commit 946aab6 with merge 05b106c...

bors-servo pushed a commit that referenced this pull request May 24, 2016
Implement PartialEq, Eq, PartialOrd, Ord on SmallVec.

Currently one can compare slices, but having the traits on the `SmallVec` itself is slightly more convenient. Perhaps there's a reason these traits aren't yet implemented, but if not, here's a PR!

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-smallvec/22)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - travis

@bors-servo bors-servo merged commit 946aab6 into servo:master May 24, 2016
@mbrubeck mbrubeck mentioned this pull request Jun 14, 2016
bors-servo pushed a commit that referenced this pull request Jun 14, 2016
Release version 0.1.8

Fixes #24.

Changelog:

* #22 Implement PartialEq, Eq, PartialOrd, and Ord on SmallVec

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-smallvec/25)
<!-- Reviewable:end -->
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