Remove an unused function

This function is exposed by our extension module, but never used by any
of our Python modules.

Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>

Author: godlygeek

src/pystack/_pystack.pyi

@@ -82,7 +82,6 @@ def get_process_threads_for_core(
     method: StackMethod = StackMethod.AUTO,
 ) -> List[PyThread]: ...
 def get_bss_info(binary: Union[str, pathlib.Path]) -> Optional[Dict[str, Any]]: ...
-def copy_memory_from_address(pid: int, address: int, size: int) -> bytes: ...
 def _check_interpreter_shutdown(manager: ProcessManager) -> None: ...
 def is_eval_frame(symbol: str, python_version: Tuple[int, int]) -> bool: ...
 def _normalize_threads_for_testing(

src/pystack/_pystack/bindings.cpp

@@ -363,18 +363,6 @@ class ProcessManagerWrapper
     std::shared_ptr<pystack::AbstractProcessManager> d_manager;
 };
 
-nb::bytes
-copy_memory_from_address(pid_t pid, uintptr_t address, size_t size)
-{
-    auto manager = std::make_shared<pystack::ProcessMemoryManager>(pid);
-    PyObject* py_bytes = PyBytes_FromStringAndSize(nullptr, static_cast<Py_ssize_t>(size));
-    if (!py_bytes) {
-        throw nb::python_error();
-    }
-    manager->copyMemoryFromProcess(address, size, PyBytes_AS_STRING(py_bytes));
-    return nb::steal<nb::bytes>(py_bytes);
-}
-
 nb::object
 get_bss_info(const std::filesystem::path& binary)
 {
@@ -972,13 +960,6 @@ NB_MODULE(_pystack, m)
                     nb::rv_policy::reference)
             .def("__exit__", [](ProcessManagerWrapper& self, nb::args) { self.reset(); });
 
-    m.def("copy_memory_from_address",
-          &copy_memory_from_address,
-          "pid"_a,
-          "address"_a,
-          "size"_a,
-          "Copy memory from a remote process");
-
     m.def("get_bss_info", &get_bss_info, "binary"_a, "Get BSS section information from an ELF binary");
 
     // Note: We use nb::arg().none() to allow None to be passed explicitly