Fix opening XBM images with arbitrary array names - #10088
teachershuang wants to merge 1 commit into
Conversation
Signed-off-by: teachershuang <guhongkaiailaopo@gmail.com>
|
Thanks for approving the workflows. At The PNG loader, leak test/helper and native decoder are unchanged. The same test on the PR's baseline Could a maintainer rerun the failed macOS job? My rerun attempt requires repository admin rights. Other current checks, including Codecov patch/project, pass; I am keeping this Draft pending the failing job and the reference-implementation discussion in #10087. No code or test expectations changed to hide the failure. |
|
I've rerun the failing test, and it has succeeded now. |
Fixes #10087 (the array-name report).
Changes proposed in this pull request:
_bitsparsing, and otherwise accept a C identifier for an initialized bitmap array.Validation for
58e441aon Windows, CPython 3.12.14, a native current-main core/zlib build:python3 selftest.py: 59 passed.--fail-on-warningpassed.--platform linux. The default Windows tox command reports 19 errors in 8 unchanged files.AI-assisted implementation and testing. The completed independent full-diff Codex review found no remaining actionable defects.
CI update (October 1,
58e441a): all 56 current checks succeeded, including platform tests, lint, docs, benchmarks and Codecov patch/project. A maintainer reran the macOS Python 3.13 job: the previously failing truncated-PNG leak test now passes; that job completed with 5684 passed, 48 skipped and 3 xfailed. No code or test expectations were changed to clear it. Draft still awaits maintainer direction on permissive array-name compatibility versus Xlib's stricter reader and the reporter's producer details, discussed in #10087.