fixed __cause__ on overload bind failure and array conversion

lostmsu · lostmsu · commit 650ac778b9de · 2021-04-12T18:06:39.000-07:00
diff --git a/src/runtime/converter.cs b/src/runtime/converter.cs
@@ -814,9 +814,14 @@ private static bool ToPrimitive(IntPtr value, Type obType, out object result, bo
 
         private static void SetConversionError(IntPtr value, Type target)
         {
+            // PyObject_Repr might clear the error
+            Runtime.PyErr_Fetch(out var causeType, out var causeVal, out var causeTrace);
+
             IntPtr ob = Runtime.PyObject_Repr(value);
             string src = Runtime.GetManagedString(ob);
             Runtime.XDecref(ob);
+
+            Runtime.PyErr_Restore(causeType, causeVal, causeTrace);
             Exceptions.RaiseTypeError($"Cannot convert {src} to {target}");
         }
 
diff --git a/src/runtime/methodbinder.cs b/src/runtime/methodbinder.cs
@@ -926,7 +926,9 @@ internal virtual IntPtr Invoke(IntPtr inst, IntPtr args, IntPtr kw, MethodBase i
                 }
 
                 value.Append(": ");
+                Runtime.PyErr_Fetch(out var errType, out var errVal, out var errTrace);
                 AppendArgumentTypes(to: value, args);
+                Runtime.PyErr_Restore(errType, errVal, errTrace);
                 Exceptions.RaiseTypeError(value.ToString());
                 return IntPtr.Zero;
             }

Original file line number	Diff line number	Diff line change
`@@ -814,9 +814,14 @@ private static bool ToPrimitive(IntPtr value, Type obType, out object result, bo`
`814`	`814`
`815`	`815`	`private static void SetConversionError(IntPtr value, Type target)`
`816`	`816`	`{`
	`817`	`+ // PyObject_Repr might clear the error`
	`818`	`+ Runtime.PyErr_Fetch(out var causeType, out var causeVal, out var causeTrace);`
	`819`	`+`
`817`	`820`	`IntPtr ob = Runtime.PyObject_Repr(value);`
`818`	`821`	`string src = Runtime.GetManagedString(ob);`
`819`	`822`	`Runtime.XDecref(ob);`
	`823`	`+`
	`824`	`+ Runtime.PyErr_Restore(causeType, causeVal, causeTrace);`
`820`	`825`	`Exceptions.RaiseTypeError($"Cannot convert {src} to {target}");`
`821`	`826`	`}`
`822`	`827`
Original file line number	Diff line number	Diff line change
`@@ -926,7 +926,9 @@ internal virtual IntPtr Invoke(IntPtr inst, IntPtr args, IntPtr kw, MethodBase i`
`926`	`926`	`}`
`927`	`927`
`928`	`928`	`value.Append(": ");`
	`929`	`+ Runtime.PyErr_Fetch(out var errType, out var errVal, out var errTrace);`
`929`	`930`	`AppendArgumentTypes(to: value, args);`
	`931`	`+ Runtime.PyErr_Restore(errType, errVal, errTrace);`
`930`	`932`	`Exceptions.RaiseTypeError(value.ToString());`
`931`	`933`	`return IntPtr.Zero;`
`932`	`934`	`}`