Compare commits

..

No commits in common. "c2877c80530646a5f3e4b679c9bcf347a1acf362" and "bf8557798acf53e960feaf9f80c99baa3e6d63ae" have entirely different histories.

2 changed files with 10 additions and 16 deletions

View file

@ -16,7 +16,7 @@ endif()
# Add include directories # Add include directories
include_directories(${CMAKE_SOURCE_DIR}/deps) include_directories(${CMAKE_SOURCE_DIR}/deps)
include_directories(SYSTEM ${CMAKE_SOURCE_DIR}/deps/libtcod) include_directories(${CMAKE_SOURCE_DIR}/deps/libtcod)
# Python includes: use different paths for Windows vs Linux # Python includes: use different paths for Windows vs Linux
if(MCRF_CROSS_WINDOWS) if(MCRF_CROSS_WINDOWS)
@ -29,7 +29,7 @@ if(MCRF_CROSS_WINDOWS)
include_directories(${CMAKE_SOURCE_DIR}/deps/cpython/PC) # For other Windows-specific headers include_directories(${CMAKE_SOURCE_DIR}/deps/cpython/PC) # For other Windows-specific headers
# Also include SFML and libtcod Windows headers # Also include SFML and libtcod Windows headers
include_directories(${CMAKE_SOURCE_DIR}/__lib_windows/sfml/include) include_directories(${CMAKE_SOURCE_DIR}/__lib_windows/sfml/include)
include_directories(SYSTEM ${CMAKE_SOURCE_DIR}/__lib_windows/libtcod/include) include_directories(${CMAKE_SOURCE_DIR}/__lib_windows/libtcod/include)
else() else()
# Native builds (Linux/Windows): use existing Python setup # Native builds (Linux/Windows): use existing Python setup
include_directories(${CMAKE_SOURCE_DIR}/deps/cpython) include_directories(${CMAKE_SOURCE_DIR}/deps/cpython)

View file

@ -37,16 +37,14 @@ void PythonObjectCache::registerObject(uint64_t serial, PyObject* weakref) {
PyObject* PythonObjectCache::lookup(uint64_t serial) { PyObject* PythonObjectCache::lookup(uint64_t serial) {
if (serial == 0) return nullptr; if (serial == 0) return nullptr;
// No mutex needed for read - GIL protects PyWeakref_GetRef // No mutex needed for read - GIL protects PyWeakref_GetObject
auto it = cache.find(serial); auto it = cache.find(serial);
if (it != cache.end()) { if (it != cache.end()) {
PyObject* obj = nullptr; PyObject* obj = PyWeakref_GetObject(it->second);
int result = PyWeakref_GetRef(it->second, &obj); if (obj && obj != Py_None) {
if (result == 1 && obj) { Py_INCREF(obj);
// obj is already a strong reference from PyWeakref_GetRef
return obj; return obj;
} }
// result == 0: dead reference, result == -1: error
} }
return nullptr; return nullptr;
} }
@ -67,15 +65,11 @@ void PythonObjectCache::cleanup() {
auto it = cache.begin(); auto it = cache.begin();
while (it != cache.end()) { while (it != cache.end()) {
PyObject* obj = nullptr; PyObject* obj = PyWeakref_GetObject(it->second);
int result = PyWeakref_GetRef(it->second, &obj); if (!obj || obj == Py_None) {
if (result <= 0) {
// Dead reference or error - remove from cache
Py_DECREF(it->second); Py_DECREF(it->second);
it = cache.erase(it); it = cache.erase(it);
} else { } else {
// Still alive - release the strong reference we obtained
Py_DECREF(obj);
++it; ++it;
} }
} }