FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[pull] master from pythonnet:master by pull[bot] · Pull Request #52 · QSLee-Net/pythonnet · GitHub

Repository navigation

[pull] master from pythonnet:master - #52

Merged
pull[bot] merged 2 commits into
QSLee-Net:masterfrom
pythonnet:master
Sep 25, 2026
Merged

pull[bot] merged 2 commits into
QSLee-Net:masterfrom
pythonnet:master

Conversation

pull Bot commented Sep 25, 2026 •
edited
Loading

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

macaronikazoo and others added 2 commits September 25, 2026 13:23
Converting a CLR value to Python can throw after the call has
succeeded, eg if an encoder's TryEncode throws, or if calling
enumerator.Current throws. Previously the exception was never
converted to a Python exception making it uncatchable even
instead, so Python code could not catch it, not even using
except BaseException

- MethodBinder.Invoke guarded only the invocation, and converted the
  result and out parameters after its try/catch
- Iterator.tp_iternext guarded MoveNext but not Current or the
  conversion of the item
- ArrayObject indexing converted items with no guard at all

Each now reports the failure through Exceptions.SetError, as the
existing catch already does for the call itself. Property and field
getters already convert inside their try and are unchanged.
pull Bot locked and limited conversation to collaborators Sep 25, 2026
pull Bot added the ⤵️ pull label Sep 25, 2026
pull Bot merged commit 72df9ae into QSLee-Net:master Sep 25, 2026

This branch had an error being deployed

1 failed deployment
github-pages — 72df9aeb Deployed Sep 25, 2026 by pull[bot] via deploy #135
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL