Remove unused function

Author: james-d-mitchell

libsemigroups_pybind11/action.py

@@ -57,7 +57,7 @@
     may_return_wrapped_cxx_obj as _may_return_wrapped_cxx_obj,
     register_cxx_wrapped_type as _register_cxx_wrapped_type,
     to_cxx as _to_cxx,
-    to_py_new as _to_py,
+    to_py as _to_py,
 )
 
 from .detail.decorators import copydoc as _copydoc

libsemigroups_pybind11/adapters.py

@@ -32,7 +32,7 @@
 from .detail.cxx_wrapper import (
     CxxWrapper as _CxxWrapper,
     to_cxx as _to_cxx,
-    to_py_new as _to_py,
+    to_py as _to_py,
 )
 
 from .tools import ordinal
@@ -50,8 +50,6 @@ class _ImageAction(_CxxWrapper):
     def __init__(self: _Self, *args, point=None, element=None) -> None:
         """
         TODO
-        Note to self <point> and <element> are the types of the objects used by
-        this function not examples of such objects.
         """
         super().__init__(
             *args,

libsemigroups_pybind11/detail/cxx_wrapper.py

@@ -37,20 +37,12 @@ def to_cxx(x: Any) -> Any:
     return x
 
 
-# TODO rm
-def to_py(Element: Any, x: Any, *args) -> Any:  # pylint: disable=invalid-name
+def to_py(x: Any, *args) -> Any:
     """
-    This function returns Element(x) if x is not None and type(x) != Element, and x o/w.
-    """
-    if x is not None and not isinstance(x, Element):
-        return Element(x, *args)
-    return x
-
-
-def to_py_new(x: Any, *args) -> Any:
-    """
-    This function returns Element(x) if x is not None and type(x) != Element, and x o/w.
+    This function returns the CxxWrapper type wrapping an instance of <x> if
+    <x> is an instance of CxxWrapper type, and <x> o/w.
     """
+    # TODO replace with isinstance(x, CxxWrapper)
     if type(x) in _CXX_WRAPPED_TYPE_TO_PY_TYPE:
         return _CXX_WRAPPED_TYPE_TO_PY_TYPE[type(x)](x, *args)
     return x
@@ -145,6 +137,7 @@ def __repr__(self: Self) -> str:
     def __copy__(self: Self) -> Self:
         if self._cxx_obj is not None:
             if hasattr(self._cxx_obj, "__copy__"):
+                # TODO use to_py
                 return _CXX_WRAPPED_TYPE_TO_PY_TYPE[type(self._cxx_obj)](
                     self._cxx_obj.__copy__()
                 )
@@ -246,6 +239,7 @@ def wrap_cxx_free_fn(cxx_free_fn: pybind11_type) -> Callable:
     """
 
     def cxx_free_fn_wrapper(*args):
+        # TODO use to_py
         result = cxx_free_fn(*(to_cxx(x) for x in args))
         if type(result) in _CXX_WRAPPED_TYPE_TO_PY_TYPE:
             return _CXX_WRAPPED_TYPE_TO_PY_TYPE[type(result)](result)
@@ -288,6 +282,6 @@ def may_return_wrapped_cxx_obj(method):
 
     @wraps(method)
     def wrapper(self, *args):
-        return to_py_new(method(self, *args))
+        return to_py(method(self, *args))
 
     return wrapper

libsemigroups_pybind11/froidure_pin.py

@@ -84,7 +84,7 @@
     register_cxx_wrapped_type as _register_cxx_wrapped_type,
     may_return_wrapped_cxx_obj as _may_return_wrapped_cxx_obj,
     to_cxx as _to_cxx,
-    to_py_new as _to_py,
+    to_py as _to_py,
     wrap_cxx_free_fn as _wrap_cxx_free_fn,
 )
 

libsemigroups_pybind11/konieczny.py

@@ -49,7 +49,7 @@
 
 from .detail.cxx_wrapper import (
     to_cxx as _to_cxx,
-    to_py_new as _to_py,
+    to_py as _to_py,
     CxxWrapper as _CxxWrapper,
     register_cxx_wrapped_type as _register_cxx_wrapped_type,
     copy_cxx_mem_fns as _copy_cxx_mem_fns,