Merge pull request #677 from GSPP/master

filmor · web-flow · commit 06ded7a2391a · 2018-05-28T11:41:27.000+02:00
Fix PyObject.GetHashCode
diff --git a/AUTHORS.md b/AUTHORS.md
@@ -48,4 +48,5 @@
 -   ([@rmadsen-ks](https://github.com/rmadsen-ks))
 -   ([@stonebig](https://github.com/stonebig))
 -   ([@testrunner123](https://github.com/testrunner123))
+-   ([@GSPP](https://github.com/GSPP))
 
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -31,6 +31,7 @@ This document follows the conventions laid out in [Keep a CHANGELOG][].
 -   Fixed `clr.GetClrType` when iterating over `System` members (#607) 
 -   Fixed `LockRecursionException` when loading assemblies (#627)
 -   Fixed errors breaking .NET Remoting on method invoke (#276)
+-   Fixed PyObject.GetHashCode (#676)
 
 
 ## [2.3.0][] - 2017-03-11
diff --git a/src/runtime/pyobject.cs b/src/runtime/pyobject.cs
@@ -909,7 +909,7 @@ public override bool Equals(object o)
         /// </remarks>
         public override int GetHashCode()
         {
-            return Runtime.PyObject_Hash(obj).ToInt32();
+            return ((ulong)Runtime.PyObject_Hash(obj)).GetHashCode();
         }
 
 

Original file line number	Diff line number	Diff line change
`@@ -909,7 +909,7 @@ public override bool Equals(object o)`
`909`	`909`	`/// </remarks>`
`910`	`910`	`public override int GetHashCode()`
`911`	`911`	`{`
`912`		`- return Runtime.PyObject_Hash(obj).ToInt32();`
	`912`	`+ return ((ulong)Runtime.PyObject_Hash(obj)).GetHashCode();`
`913`	`913`	`}`
`914`	`914`
`915`	`915`