fixed last todo

sagemath · Sep 19, 2018 · dd429d5 · dd429d5
1 parent a5d0e7f
commit dd429d5
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/src/sage/libs/gap/element.pyx b/src/sage/libs/gap/element.pyx
@@ -2814,10 +2814,7 @@ cdef class GapElement_Permutation(GapElement):
         if parent is None:
             return Permutation(lst.sage(), check_input=False)
         else:
-            # TODO: test that the parent is fine
-            one = parent.one()
-            one_c = one
-            return one_c._generate_new_GAP(lst)
+            return parent.one()._generate_new_GAP(lst)
 
 ############################################################################
 ### GapElement_Record ######################################################