Skip to content

fix rust linked list #1609

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
Merged
merged 1 commit into from
Jan 13, 2025
Merged

fix rust linked list #1609

merged 1 commit into from
Jan 13, 2025
+40 −24

Conversation

rongyi
Copy link
Contributor

@rongyi rongyi commented Jan 8, 2025

rust linked_list update

  1. in remove, the if let check already exclude the None case, so no need in rust
  2. in find, the third argument is only show in rust, the signature should be API level equivalence. hide this in inner function.

happy 2025!

@krahets
Copy link
Owner

krahets commented Jan 13, 2025

Happy new year @rongyi !

@krahets krahets added code Code-related polish Decorative detail or feature labels Jan 13, 2025
@krahets krahets merged commit 7503a33 into krahets:main Jan 13, 2025
4 checks passed
This was referenced Jun 30, 2025
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krahets krahets krahets approved these changes

Assignees
No one assigned
Labels
code Code-related polish Decorative detail or feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rongyi @krahets