🌐 AI搜索 & 代理 主页
Skip to content

Better error messages for method argument mismatch and others #1361

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lostmsu merged 5 commits into from
Jan 27, 2021
Merged

Better error messages for method argument mismatch and others #1361

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
716c98d
Better error messages for method argument mismatch, Python to managed…
tminka Jan 22, 2021
94b0c35
PythonException.Normalize throws if an error is set
tminka Jan 26, 2021
914f0e6
ExceptionClassObject.tp_str calls Exception.ToString() instead of Exc…
tminka Jan 26, 2021
1caded0
Revert change to GetAggregateException
tminka Jan 26, 2021
b8a7b9e
Disabled test_method_parameters_change
tminka Jan 27, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert change to GetAggregateException
  • Loading branch information
@tminka
tminka committed Jan 26, 2021
commit 1caded078c1e7d5ac4cadd4d767c182810ffe00a
3 changes: 2 additions & 1 deletion src/runtime/methodbinder.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -560,7 +560,8 @@ internal Binding Bind(IntPtr inst, IntPtr args, IntPtr kw, MethodBase info, Meth

static AggregateException GetAggregateException(IEnumerable<MismatchedMethod> mismatchedMethods)
{
return new AggregateException(mismatchedMethods.Select(m => new ArgumentException($"{m.Exception.Message} in method {m.Method}", m.Exception)));
// We cannot attach m.Exception as an InnerException here because it contains pointers to Python objects and therefore not serializable.
return new AggregateException(mismatchedMethods.Select(m => new ArgumentException($"{m.Exception.Message} in method {m.Method}")));
}

static IntPtr HandleParamsArray(IntPtr args, int arrayStart, int pyArgCount, out bool isNewReference)
Expand Down