diff options
| author | Dan Gohman <gohman@apple.com> | 2010-09-01 14:00:35 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2010-09-01 14:00:35 +0000 |
| commit | e3955df639ff9aff990f628ef6a219ff5efdbc81 (patch) | |
| tree | 59fc80421919816021622260f4ae70d1ea434c12 /include/llvm/ADT/DenseSet.h | |
| parent | f8ff40c0598c6d8707a639ec63644e5e5496678a (diff) | |
| download | external_llvm-e3955df639ff9aff990f628ef6a219ff5efdbc81.zip external_llvm-e3955df639ff9aff990f628ef6a219ff5efdbc81.tar.gz external_llvm-e3955df639ff9aff990f628ef6a219ff5efdbc81.tar.bz2 | |
Make the iterator form of erase return void, since it always succeeds,
and since this is what std::map and std::set do.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@112701 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/llvm/ADT/DenseSet.h')
| -rw-r--r-- | include/llvm/ADT/DenseSet.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/llvm/ADT/DenseSet.h b/include/llvm/ADT/DenseSet.h index 765105b..00bcf64 100644 --- a/include/llvm/ADT/DenseSet.h +++ b/include/llvm/ADT/DenseSet.h @@ -106,8 +106,8 @@ public: const_iterator end() const { return ConstIterator(TheMap.end()); } iterator find(const ValueT &V) { return Iterator(TheMap.find(V)); } - bool erase(Iterator I) { return TheMap.erase(I.I); } - bool erase(ConstIterator CI) { return TheMap.erase(CI.I); } + void erase(Iterator I) { return TheMap.erase(I.I); } + void erase(ConstIterator CI) { return TheMap.erase(CI.I); } std::pair<iterator, bool> insert(const ValueT &V) { return TheMap.insert(std::make_pair(V, 0)); |