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

Move semantics for StringHashMap #589

Merged

Conversation

@rsKliPPy
Copy link
Contributor

rsKliPPy commented Feb 22, 2017

StringHashMap<T> couldn't previously contain non-copyable objects, like ke::AutoPtr<T> in my case.
This PR fixes that and adds new bool retrieve(const char *, T **) for retrieving a reference to an object within the container in case it's not copyable.

@rsKliPPy rsKliPPy force-pushed the rsKliPPy:stringhashmap-move-semantics branch 2 times, most recently from c72f198 to 379501b Feb 22, 2017
@rsKliPPy rsKliPPy force-pushed the rsKliPPy:stringhashmap-move-semantics branch from 379501b to 377d56c Feb 22, 2017
@KyleSanderson KyleSanderson merged commit a81b52a into alliedmodders:master Jul 12, 2018
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@KyleSanderson

This comment has been minimized.

Copy link
Member

KyleSanderson commented Jul 12, 2018

apologies for the delay and thank you.

@rsKliPPy rsKliPPy deleted the rsKliPPy:stringhashmap-move-semantics branch Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.