-
Notifications
You must be signed in to change notification settings - Fork 59
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
Ready for release? #28
Comments
@rickb777 I have made a few changes that I think make it ready for release.
q.Grow(len(elems))
for i := range elems {
q.PushBack(elems[i])
}
I am considering an for i := 0; i < q.Len(); i++ {
doSomething(q.At(i))
} This could also include for q.Len() != 0 {
process(q.PopFront())
} The iterator looks like:
Probably worth adding iterators to eliminate intermediate resizes during iteration. Somewhat like the inverse of As for a |
Completed. Closing this PR as v1.0.0 is now released. Discussion continued in #35. |
Deque is a good utility, clear and simple to use, and fast.
Is it ready for a release version? It's overdue for one, IMO :-)
Meanwhile, here are some ideas for methods that you could add:
Full() bool
method - a shortcut forq.Len() == q.Cap()
which would be useful for anyone wanting to impose a maximum size or avoid resizingThe text was updated successfully, but these errors were encountered: