std::map::erase() can take a key directly and has no-throw guarantee.

diff --git a/taglib/toolkit/tmap.tcc b/taglib/toolkit/tmap.tcc
index 5d3abcad5010b30a7793085fe83a34c9335518dc..8c1106dfc53b452c590d550d881a3d36165328b8 100644 (file)
--- a/taglib/toolkit/tmap.tcc
+++ b/taglib/toolkit/tmap.tcc
@@ -145,9 +145,7 @@ template <class Key, class T>
 Map<Key, T> &Map<Key,T>::erase(const Key &key)
 {
   detach();
-  Iterator it = d->map.find(key);
-  if(it != d->map.end())
-    d->map.erase(it);
+  d->map.erase(key);
   return *this;
 }