Skip to content

Improve method binding #974

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

Closed
wants to merge 11 commits into from
Closed

Improve method binding #974

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
21528ee
Add test for ienumerable method
koubaa Oct 21, 2019
b4d60c6
Add tests for collection & task/action from function
koubaa Dec 1, 2019
3f02abf
Add more tests
koubaa Dec 1, 2019
9c3e1c5
Implement list converter
koubaa Dec 1, 2019
b9db0f6
Implement action converter
koubaa Dec 1, 2019
4b0a1ad
add TODO
koubaa Dec 1, 2019
8bed1bb
Update changelog
koubaa Dec 1, 2019
a4599d9
share common code between ToArray and ToList
koubaa Dec 2, 2019
96fd896
improve ref counting
koubaa Dec 3, 2019
c6fd768
revert whitespace
koubaa Dec 4, 2019
e63d11e
list wrapper
koubaa Dec 8, 2019
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 whitespace
  • Loading branch information
@koubaa
koubaa committed Dec 4, 2019
commit c6fd7686935f5d20d41ca1f14202514dbb7cb9f2
10 changes: 6 additions & 4 deletions src/runtime/converter.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -878,14 +878,16 @@ private static bool ToListOfT(IntPtr value, Type elementType, out object result)

var listType = typeof(List<>);
var constructedListType = listType.MakeGenericType(elementType);
IList list = IsSeqObj ? (IList)Activator.CreateInstance(constructedListType, new Object[] { (int)len }) :
(IList)Activator.CreateInstance(constructedListType);
IList list = IsSeqObj ? (IList) Activator.CreateInstance(constructedListType, new Object[] { (int)len }) :
(IList) Activator.CreateInstance(constructedListType);
IntPtr item;

while ((item = Runtime.PyIter_Next(IterObject)) != IntPtr.Zero) {
while ((item = Runtime.PyIter_Next(IterObject)) != IntPtr.Zero)
{
object obj = null;

if (!Converter.ToManaged(item, elementType, out obj, true)) {
if (!Converter.ToManaged(item, elementType, out obj, true))
{
Runtime.XDecref(item);
return false;
}
Expand Down