Remove unnec. decorator

Author: james-d-mitchell

libsemigroups_pybind11/action.py

@@ -54,7 +54,6 @@
 from .detail.cxx_wrapper import (
     CxxWrapper as _CxxWrapper,
     copy_cxx_mem_fns as _copy_cxx_mem_fns,
-    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 as _to_py,
@@ -248,9 +247,8 @@ def __init__(
         for x in seeds:
             self.add_seed(x)
 
-    @_may_return_wrapped_cxx_obj
     def __getitem__(self: _Self, i: int) -> Point:
-        return _to_cxx(self)[i]
+        return _to_py(_to_cxx(self)[i])
 
     def __len__(self: _Self) -> int:
         return _to_cxx(self).size()

libsemigroups_pybind11/detail/cxx_wrapper.py

@@ -233,24 +233,3 @@ def copy_cxx_mem_fns(cxx_class: pybind11_type, py_class: CxxWrapper) -> None:
                 py_meth_name,
                 wrap_cxx_mem_fn(getattr(cxx_class, py_meth_name)),
             )
-
-
-########################################################################
-# Decorators
-########################################################################
-
-
-# TODO remove usage of this just call to_py_new directly
-def may_return_wrapped_cxx_obj(method):
-    """
-    Decorator for methods that might return a wrapped C++ object. If a wrapped
-    C++ object is returned, then it wrapped appropriately before being
-    returned. This is usually handled automatically above, except for
-    special methods, which are sometimes declared explicitly.
-    """
-
-    @wraps(method)
-    def wrapper(self, *args):
-        return to_py(method(self, *args))
-
-    return wrapper

libsemigroups_pybind11/froidure_pin.py

@@ -82,7 +82,6 @@
     CxxWrapper as _CxxWrapper,
     copy_cxx_mem_fns as _copy_cxx_mem_fns,
     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 as _to_py,
     wrap_cxx_free_fn as _wrap_cxx_free_fn,
@@ -174,10 +173,9 @@ def __init__(self: Self, *args) -> None:
         self.py_template_params = (type(gens[0]),)
         self.init_cxx_obj(gens)
 
-    @_may_return_wrapped_cxx_obj
     def __getitem__(self: Self, i: int) -> Element:
         self._raise_if_element_not_implemented()
-        return _to_cxx(self)[i]
+        return _to_py(_to_cxx(self)[i])
 
     def __iter__(self: Self) -> Iterator:
         self._raise_if_element_not_implemented()