graph: Fix bug in CacheWeight.weight

lutter · lutter · commit 5c25b9ac7378 · 2021-11-29T13:56:43.000-08:00
The bug made any struct have the size of a pointer, i.e., 8 bytes on 64 bit
machines.
diff --git a/graph/src/util/cache_weight.rs b/graph/src/util/cache_weight.rs
@@ -9,7 +9,7 @@ use std::{collections::BTreeMap, mem};
 pub trait CacheWeight {
     /// Total weight of the value.
     fn weight(&self) -> usize {
-        mem::size_of_val(&self) + self.indirect_weight()
+        mem::size_of_val(self) + self.indirect_weight()
     }
 
     /// The weight of values pointed to by this value but logically owned by it, which is not

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ use std::{collections::BTreeMap, mem};`
`9`	`9`	`pub trait CacheWeight {`
`10`	`10`	`/// Total weight of the value.`
`11`	`11`	`fn weight(&self) -> usize {`
`12`		`- mem::size_of_val(&self) + self.indirect_weight()`
	`12`	`+ mem::size_of_val(self) + self.indirect_weight()`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`/// The weight of values pointed to by this value but logically owned by it, which is not`