Files
mtgodot-poc/audit/port-map/ScriptLib/PythonMarshal.cpp.json
T
shenleiandClaude Opus 5 43bb561551 port(2P step 2): ScriptLib/PythonLauncher on the embedded interpreter
Link mtpython into port_logic and copy 40250's ScriptLib launcher across:
PythonLauncher, PythonUtils and PythonMarshal, plus the 12 forwarding headers
that make its `#include <Python-2.7/*>` lines compile unchanged. symtable.h
forward-declares mod_ty rather than including Python-ast.h, whose one-word
macros (Delete, Module, Set, Print) collide with EterLib/Pool.h — the same
clash 40250 sidestepped by commenting out PySymtable_Build.

Two `// PORT:` edits: `unsigned char(val)` is an MSVC extension (PythonUtils.cpp,
twice), and PyObject_AsCharBuffer wants a Py_ssize_t* (TraceFunc).

port_python_launcher_test drives the launcher in the order UserInterface.cpp
does (:241-434): Create(), __DEBUG__/__COMMAND_LINE__, TRUE/FALSE, a CRLF script
through RunMemoryTextFile, a .pyc through RunCompiledFile, then RunFile reading
system.py out of the real pack. It stops at `ImportError: No module named app`,
i.e. the first C++ module RunMainScript registers — that is the 2V0 slice, and
the stdlib behind it is step 3. The host flags (Py_NoSiteFlag and friends) sit
in the test for now; step 3 moves them into the platform layer.

port-map statuses are set only for what the test actually executes: 10 of 12
launcher functions and 4 of 11 marshal ones (the small-.pyc path delegates to
CPython's own reader, so the ported r_object stays unreached). PythonUtils
compiles but nothing calls it until the binding modules arrive, so it stays TODO.

Windows has no mtpython yet, so port_logic drops ScriptLib and the shims there.

Tests: macOS ctest 27/27 incl. port.python_launcher; port_logic (with ScriptLib)
also compiles for android arm64 (NDK API 24) and ios arm64.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-23 08:54:48 +09:00

58 lines
1.6 KiB
JSON

{
"reference": "ScriptLib/PythonMarshal.cpp",
"reference_sha256": "c12ee08f78c9070097b851a5d0a22be9999e12fc03e04c6da639c5f1bbd81f05",
"priority": "P4",
"contracts": [],
"functions": {
"r_string": {
"status": "TODO"
},
"r_short": {
"status": "TODO"
},
"r_long": {
"impl": [
"extension/src/port/ScriptLib/PythonMarshal.cpp:r_long"
],
"status": "PORTED",
"test": "extension/tests/port_python_launcher_test.cpp (port.python_launcher)"
},
"r_long64": {
"status": "TODO"
},
"r_object": {
"status": "TODO"
},
"_PyMarshal_ReadShortFromFile": {
"status": "TODO"
},
"_PyMarshal_ReadLongFromFile": {
"impl": [
"extension/src/port/ScriptLib/PythonMarshal.cpp:_PyMarshal_ReadLongFromFile"
],
"status": "PORTED",
"test": "extension/tests/port_python_launcher_test.cpp (port.python_launcher)"
},
"getfilesize": {
"impl": [
"extension/src/port/ScriptLib/PythonMarshal.cpp:getfilesize"
],
"status": "PORTED",
"test": "extension/tests/port_python_launcher_test.cpp (port.python_launcher)"
},
"_PyMarshal_ReadLastObjectFromFile": {
"impl": [
"extension/src/port/ScriptLib/PythonMarshal.cpp:_PyMarshal_ReadLastObjectFromFile"
],
"status": "PORTED",
"test": "extension/tests/port_python_launcher_test.cpp (port.python_launcher)"
},
"_PyMarshal_ReadObjectFromFile": {
"status": "TODO"
},
"_PyMarshal_ReadObjectFromString": {
"status": "TODO"
}
}
}