Skip to content

add various missing utility functions - #51

Merged
djc merged 1 commit into
djc:mainfrom
AlexanderSchuetz97:main
Jul 2, 2026
Merged

add various missing utility functions#51
djc merged 1 commit into
djc:mainfrom
AlexanderSchuetz97:main

Conversation

@AlexanderSchuetz97

@AlexanderSchuetz97 AlexanderSchuetz97 commented Jun 21, 2026

Copy link
Copy Markdown
Contributor
  • insert_entry to RawVacantEntryMut and VacantEntry
  • front_entry/back_entry to map -> Option RawOccupiedEntryMut
  • current_entry to CursorMut -> Result RawOccupiedEntryMut Self

solves #50

I also added all the other gadgets that I also found absent that I require for my use case.
I currently have code that uses BTreeMap that I want to change to LinkedHashMap because I need to re-order
the map based on external criteria. I like your Cursor struct btw. its good.

Let me know what you think

@AlexanderSchuetz97

Copy link
Copy Markdown
Contributor Author

@djc can you review this?

@djc

djc commented Jun 22, 2026

Copy link
Copy Markdown
Owner

@djc can you review this?

No need to remind me within 24 hours.

@AlexanderSchuetz97

Copy link
Copy Markdown
Contributor Author

sorry, wasn't sure github notfied you that this PR exists. I don't get emails for pull requests on my repos unless someone assign me. But I was not able to assign you so I tagged you. If you got notified of this PR by itself then you must have done some settings in gh that I couldn't find.

Sorry for bothering you. Take your time.

@djc

djc commented Jun 22, 2026

Copy link
Copy Markdown
Owner

There's a Watch button on the repo front page that people can use to be notified of all repo activity.

@djc

djc commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Can you address the CI failure, and squash your commits? Otherwise this looks okay, thanks!

- insert_entry to RawVacantEntryMut and VacantEntry
- front_entry/back_entry to map -> Option<RawOccupiedEntryMut>
- current_entry to CursorMut -> Option<RawOccupiedEntryMut>
@AlexanderSchuetz97

Copy link
Copy Markdown
Contributor Author

I hope its good now, if not then simply ping me again.
Thanks!

@djc
djc merged commit 9f998f8 into djc:main Jul 2, 2026
11 checks passed
@djc

djc commented Jul 2, 2026

Copy link
Copy Markdown
Owner

If you'd like a release, please submit a PR bumping the version to 0.12.1.

@AlexanderSchuetz97

Copy link
Copy Markdown
Contributor Author

Will do so this weekend, thanks for merging.

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.

2 participants