Skip to content
This repository was archived by the owner on Feb 1, 2025. It is now read-only.

CircularBuffer implemented - #1

Open
0xtema-m wants to merge 13 commits into
mainfrom
develop
Open

0xtema-m wants to merge 13 commits into
mainfrom
develop

Conversation

@0xtema-m

Copy link
Copy Markdown
Owner

Described all fields, method and functions to require std::random_access_iterator

Tema Moiseenko added 13 commits March 6, 2023 01:45
Described all fields, method and functions to require std::random_access_iterator
1. Some tests have been rewritten according to new for me asserts in google tests.

2. Naming mistake with CircualrBufferTraits -> CircularBufferBase

3. Instead of using private inheritance to avoid allocation of empty structure use [[no_unique_address]] fields.

4. bug fix with insert method

5. reserve_if_full method has been added.

6. Initializer lists have been rewritten according to google code style

7. All constructors in CircularBufferBase have been moved to protected section to  protected section to prevent construction of it
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant