Skip to content

Disable implicit seq to array conversion when target type is object #1901

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.

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 17 commits into from

Conversation

lostmsu
Copy link
Member

@lostmsu lostmsu commented Aug 8, 2022

What does this implement/fix? Explain your changes.

Disabled implicit conversion from Python objects implementing sequence protocol to .NET arrays when the target .NET type is System.Object. The conversion is still attempted when the target type is a System.Array.

Many Python types implement sequence protocol. In many cases such conversion looses too much important data.

Does this close any currently open issues?

fixes #1900

Checklist

Check all those that are applicable and complete.

  • Make sure to include one or more tests for your change
  • Updated the CHANGELOG

filmor and others added 17 commits June 27, 2022 11:47
fixed unhandled Python errors during comparison attempts

fixes pythonnet#1848
Additionally, fixes a type hint and makes sure that the new default
behaviour is to use the environment variable if given.
- Only include src/runtime, no other .NET subproject
- In particular, tests are not contained anymore in the sdist
- Fix accidentally including obj and bin directories in the sdist
* Add unit tests for (U)IntPtr conversions
* Special-case construction of (U)IntPtr
* Check (U)IntPtr size explicitly
…ng sequence protocol to

.NET arrays when the target .NET type is `System.Object`. The conversion is still attempted when the
target type is a `System.Array`

fixes pythonnet#1900
@lostmsu
Copy link
Member Author

lostmsu commented Aug 8, 2022

Wrong branch

@lostmsu lostmsu closed this Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot convert to managed type
2 participants