Skip to content

Commit

Permalink
COMMON: Remove return from void PtrMap::erase()
Browse files Browse the repository at this point in the history
Yeah, that method shouldn't try to return a value.
  • Loading branch information
DrMcCoy committed Oct 29, 2016
1 parent 0dfb050 commit 3b2c239
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/common/ptrmap.h
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ class PtrMap : boost::noncopyable, public std::map<Key, T *, Compare> {

void erase(typename std::map<Key, T *, Compare>::iterator position) {
Deallocator::destroy(position->second);
return std::map<Key, T *, Compare>::erase(position);
std::map<Key, T *, Compare>::erase(position);
}

typename std::map<Key, T *, Compare>::size_type
Expand Down

0 comments on commit 3b2c239

Please sign in to comment.