Relocate vm._hash to obj.hash

Author: ChJR

vm/src/builtins/genericalias.rs

@@ -167,7 +167,7 @@ fn make_parameters(args: &PyTupleRef, vm: &VirtualMachine) -> PyTupleRef {
 impl Hashable for PyGenericAlias {
     #[inline]
     fn hash(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<hash::PyHash> {
-        Ok(vm._hash(zelf.origin.as_object())? ^ vm._hash(zelf.args.as_object())?)
+        Ok(zelf.origin.as_object().hash(vm)? ^ zelf.args.as_object().hash(vm)?)
     }
 }
 

vm/src/builtins/weakref.rs

@@ -93,7 +93,7 @@ impl Hashable for PyWeak {
                 let obj = zelf
                     .upgrade()
                     .ok_or_else(|| vm.new_type_error("weak object has gone away".to_owned()))?;
-                let hash = vm._hash(&obj)?;
+                let hash = obj.hash(vm)?;
                 zelf.hash.store(Some(hash));
                 Ok(hash)
             }

vm/src/dictdatatype.rs

@@ -652,7 +652,7 @@ pub trait DictKey: IntoPyObject {
 /// to index dictionaries.
 impl DictKey for PyObjectRef {
     fn key_hash(&self, vm: &VirtualMachine) -> PyResult<HashValue> {
-        vm._hash(self)
+        self.hash(vm)
     }
 
     fn key_is(&self, other: &PyObjectRef) -> bool {

vm/src/stdlib/builtins.rs

@@ -337,7 +337,7 @@ mod builtins {
 
     #[pyfunction]
     fn hash(obj: PyObjectRef, vm: &VirtualMachine) -> PyResult<PyHash> {
-        vm._hash(&obj)
+        obj.hash(vm)
     }
 
     // builtin_help

vm/src/stdlib/sre.rs

@@ -517,7 +517,7 @@ mod _sre {
 
     impl Hashable for Pattern {
         fn hash(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<PyHash> {
-            let hash = vm._hash(&zelf.pattern)?;
+            let hash = zelf.pattern.hash(vm)?;
             let (_, code, _) = unsafe { zelf.code.align_to::<u8>() };
             let hash = hash ^ vm.state.hash_secret.hash_bytes(code);
             let hash = hash ^ (zelf.flags.bits() as PyHash);

vm/src/utils.rs

@@ -75,14 +75,14 @@ pub fn hash_iter<'a, I: IntoIterator<Item = &'a PyObjectRef>>(
     iter: I,
     vm: &VirtualMachine,
 ) -> PyResult<rustpython_common::hash::PyHash> {
-    vm.state.hash_secret.hash_iter(iter, |obj| vm._hash(obj))
+    vm.state.hash_secret.hash_iter(iter, |obj| obj.hash(vm))
 }
 
 pub fn hash_iter_unordered<'a, I: IntoIterator<Item = &'a PyObjectRef>>(
     iter: I,
     vm: &VirtualMachine,
 ) -> PyResult<rustpython_common::hash::PyHash> {
-    rustpython_common::hash::hash_iter_unordered(iter, |obj| vm._hash(obj))
+    rustpython_common::hash::hash_iter_unordered(iter, |obj| obj.hash(vm))
 }
 
 // TODO: find a better place to put this impl