From 95de32aa7fbb1a2da547418b296f649ee4be1feb Mon Sep 17 00:00:00 2001 From: Tomi Valkeinen Date: Tue, 5 Sep 2017 11:01:17 +0300 Subject: py: drop the fancy event handling Unfortunately the nice event handler added previously doesn't work: we may get multiple page-flip events, which would lead to unref'ing the passed python object multiple times, leading to memory corruption. I guess it's only possible to pass a plain int as user data to commit() and page_flip(). --- py/pykms/pykms.cpp | 5 ----- 1 file changed, 5 deletions(-) (limited to 'py/pykms/pykms.cpp') diff --git a/py/pykms/pykms.cpp b/py/pykms/pykms.cpp index 7752f19..1e54c9b 100644 --- a/py/pykms/pykms.cpp +++ b/py/pykms/pykms.cpp @@ -20,11 +20,6 @@ PYBIND11_PLUGIN(pykms) { init_pykmsbase(m); - m.def("__ob_unpack_helper", [](uint64_t v) { - // AtomicReq::commit or Crtc::page_flip added a ref, so we can use borrowed = false - return py::object((PyObject*)v, false); - }); - init_pykmstest(m); init_pyvid(m); -- cgit v1.2.3