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.

Sign up for GitHub

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

Jump to bottom

feat(collection): [#29] Allow for any kind of Index #40

Merged
merged 2 commits into from
May 14, 2023
Merged

feat(collection): [#29] Allow for any kind of Index #40

merged 2 commits into from
May 14, 2023

Conversation

pjechris
Copy link
Owner

@pjechris pjechris commented May 14, 2023 •
edited
Loading

⚽️ Description

Lift restriction on MutableCollection.Index to be an Int. Now only requirement is Index being Hashable.

🔨 Implementation details

  • Position to update value in raw buffer is now computed using distance(from:to:)
  • Update README to remove section about limitations on collections

We can now close #29

@pjechris pjechris merged commit 71cf0df into main May 14, 2023
@pjechris pjechris deleted the feat/29-collection-anyIndex branch May 14, 2023 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

💡 Lift custom collection limits
1 participant
@pjechris