Changing the hashing in attributes to what Nodes do, i.e. hash on the id of the node-value. This keeps coming up so we should rename the .hash() method in the node-value to something else. Morgan, feel free to change, but I had to go in as we were stuck on infinite parsing again.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/expr/attribute.h b/src/expr/attribute.h
index c56725f18..d7514d50c 100644
--- a/src/expr/attribute.h
+++ b/src/expr/attribute.h
@@ -46,7 +46,7 @@ namespace attr {
 struct AttrHashFcn {
   enum { LARGE_PRIME = 32452843ul };
   std::size_t operator()(const std::pair<uint64_t, NodeValue*>& p) const {
-    return p.first * LARGE_PRIME + p.second->hash();
+    return p.first * LARGE_PRIME + p.second->getId();
   }
 };
 
@@ -57,7 +57,7 @@ struct AttrHashFcn {
  */
 struct AttrHashBoolFcn {
   std::size_t operator()(NodeValue* nv) const {
-    return nv->hash();
+    return (size_t)nv->getId();
   }
 };