Skip to content

feat: add proper comparisons - #786

Open
alejandro-vaz wants to merge 6 commits into
servo:v2from
alejandro-vaz:proper-comparisons
Open

alejandro-vaz wants to merge 6 commits into
servo:v2from
alejandro-vaz:proper-comparisons

Conversation

@alejandro-vaz

@alejandro-vaz alejandro-vaz commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator

this PR completes the equivalency and comparison implementations of smallvec:

  • it adds missing PartialEq and PartialOrd implementations
  • it moves Equivalent to comparisons.rs
  • it implements Comparable where appropriate
  • it improves the macro for automatic implementations
  • it fixes one small deprecation warning

@alejandro-vaz alejandro-vaz self-assigned this Oct 21, 2026
@alejandro-vaz

alejandro-vaz commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator Author

probably one of the best PRs I've opened in my entire life

I'll write description tomorrow, it's almost 4am

@alejandro-vaz
alejandro-vaz requested a review from jdm October 21, 2026 01:41
@alejandro-vaz
alejandro-vaz marked this pull request as ready for review October 21, 2026 01:41
@alejandro-vaz

Copy link
Copy Markdown
Collaborator Author

@jdm ready for review

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.

1 participant