WaE: Unreachable code

Not sure if the intent is that clearCache() should be called, but the
calls have just accidentally been placed after the return statements,
or whether the code works correctly and as intended as it is now,
i.e. with the unreachable calls to clearCache().

Warning emitted by VS 2013, why don't other compilers warn? Perhaps
these functions are never called?

Change-Id: I16b0bc1b836753ef5579dadba211cbce4f50cbc3

1 files changed, 2 insertions, 2 deletions

diff --git a/configmgr/source/nodemap.hxx b/configmgr/source/nodemap.hxx
index 068a471c670f..dcfac2bc4441 100644
--- a/configmgr/source/nodemap.hxx
+++ b/configmgr/source/nodemap.hxx
@@ -52,8 +52,8 @@ class NodeMap
     iterator end() { return maImpl.end(); }
     const_iterator end() const { return maImpl.end(); }
 
-    rtl::Reference<Node> &operator[](const OUString &aStr) { return maImpl[aStr]; clearCache(); }
-    std::pair<iterator,bool> insert(const value_type &vt) { return maImpl.insert(vt); clearCache(); }
+    rtl::Reference<Node> &operator[](const OUString &aStr) { return maImpl[aStr]; /* clearCache(); */ }
+    std::pair<iterator,bool> insert(const value_type &vt) { return maImpl.insert(vt); /* clearCache(); */ }
     void clear() { maImpl.clear(); clearCache(); }
     void erase(const iterator &it) { maImpl.erase(it); clearCache(); }
     void erase(const OUString &aStr) { maImpl.erase(aStr); clearCache(); }