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

CCRingBuffer extensions #127

Closed
copy opened this issue May 13, 2017 · 2 comments
Closed

CCRingBuffer extensions #127

copy opened this issue May 13, 2017 · 2 comments
Assignees
@c-cube
Labels
enhancement
Milestone
1.3

Comments

@copy
Copy link
Contributor

copy commented May 13, 2017

The following would be useful:

  • is_full
  • option-returning variants of peek_front, peek_back (preferably backwards-incompatible)
@c-cube c-cube self-assigned this May 13, 2017
@c-cube c-cube added this to the 1.3 milestone May 13, 2017
@c-cube
Copy link
Owner

c-cube commented May 13, 2017

Compatibility shall be maintained, sorry. But option-returning variants are totally possible.

@c-cube c-cube mentioned this issue Jun 14, 2017
Closed
@c-cube
Copy link
Owner

c-cube commented Jun 14, 2017

Change of mind: breaking compatibility a bit, with _exn variants, is totally possible and should be part of the effort in #134. is_full is quite easy to add, too.

@c-cube c-cube closed this as completed in a3ff9db Jun 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@c-cube c-cube

Labels
enhancement
Projects
None yet
Milestone
1.3
Development

No branches or pull requests
2 participants
@c-cube @copy