Skip to content

Mods #1

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Mods #1

wants to merge 5 commits into from

Conversation

marc1uk
Copy link
Contributor

@marc1uk marc1uk commented Sep 2, 2021

  • Add GetMap and Erase methods to Store class.
  • Return 0 on Store::Get if we tried to get with an unsuitable type (i.e. tried to get a string value into a numeric variable).
  • Update the internally allocated heap object on subsequent Get calls using a pointer, to ensure the same result is returned whether one uses Store::Get(std::string key, T* val) or Store::Get(std::string key, T val).

@marc1uk marc1uk force-pushed the mods branch 2 times, most recently from bcd5795 to 5ea603f Compare August 14, 2023 20:27
@brichards64
Copy link
Contributor

check if relavent if not close

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