Fix broken 64bit hash in tbb

Fixes #181
borglab · Dec 10, 2019 · 1b391cd · 1b391cd
1 parent 60d820e
commit 1b391cd
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/gtsam/base/ConcurrentMap.h b/gtsam/base/ConcurrentMap.h
@@ -29,14 +29,22 @@
 #  undef max
 #  undef ERROR
 
+#include <functional> // std::hash()
+
 // Use TBB concurrent_unordered_map for ConcurrentMap
-#  define CONCURRENT_MAP_BASE tbb::concurrent_unordered_map<KEY, VALUE>
+template <typename KEY, typename VALUE>
+using CONCURRENT_MAP_BASE = tbb::concurrent_unordered_map<
+  KEY,
+  VALUE,
+  std::hash<KEY>
+  >;
 
 #else
 
 // If we're not using TBB, use a FastMap for ConcurrentMap
-#  include <gtsam/base/FastMap.h>
-#  define CONCURRENT_MAP_BASE gtsam::FastMap<KEY, VALUE>
+#include <gtsam/base/FastMap.h>
+template <typename KEY, typename VALUE>
+using CONCURRENT_MAP_BASE = gtsam::FastMap<KEY, VALUE>;
 
 #endif
 
@@ -57,11 +65,11 @@ namespace gtsam {
  * @addtogroup base
  */
 template<typename KEY, typename VALUE>
-class ConcurrentMap : public CONCURRENT_MAP_BASE {
+class ConcurrentMap : public CONCURRENT_MAP_BASE<KEY,VALUE> {
 
 public:
 
-  typedef CONCURRENT_MAP_BASE Base;
+  typedef CONCURRENT_MAP_BASE<KEY,VALUE> Base;
 
   /** Default constructor */
   ConcurrentMap() {}