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.

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

Iterator: general refactoring and reset method #193

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

donutloop
Copy link
Collaborator

@donutloop donutloop commented Feb 28, 2024

Major break

Feature

Reset allows for the iteration process over a sequence to be restarted from the beginning. It enables reusing the iterator for multiple traversals without needing to recreate it.

Refactoring

It is a idiomatic practice to design functions and methods to return concrete struct types. This approach promotes flexibility and decoupling, allowing the calling code to work with any implementation that satisfies the interface

@donutloop
Copy link
Collaborator Author

@duke-git Please review

@donutloop donutloop force-pushed the feature/iteratorV2 branch 2 times, most recently from 935d190 to f48947c Compare February 28, 2024 21:23
Feature

Reset allows for the iteration process over a sequence to be restarted from the beginning.
It enables reusing the iterator for multiple traversals without needing to recreate it.

Refactoring

It is a idiomatic practice to design functions and methods to return concrete struct types.
This approach promotes flexibility and decoupling, allowing the calling code to work with any implementation that satisfies the interface
@duke-git
Copy link
Owner

@duke-git Please review

@donutloop Yes, I created the rc branch for release candidate. Please merge all PRs into the rc branch. The main branch will be frozen.

@duke-git duke-git merged commit 473f9c9 into duke-git:main Feb 29, 2024
3 checks passed
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