fix suggested details

Author: fchapoton

src/sage/algebras/steenrod/steenrod_algebra.py

@@ -517,7 +517,7 @@ def __classcall__(self, p=2, basis='milnor', **kwds):
         return super().__classcall__(self, p=p, basis=std_basis, profile=std_profile,
                                      truncation_type=std_type, generic=std_generic)
 
-    def __init__(self, p=2, basis='milnor', **kwds):
+    def __init__(self, p=2, basis='milnor', **kwds) -> None:
         r"""
         INPUT:
 
@@ -692,7 +692,7 @@ def prime(self):
         """
         return self._prime
 
-    def basis_name(self):
+    def basis_name(self) -> str:
         r"""
         The basis name associated to ``self``.
 
@@ -707,7 +707,7 @@ def basis_name(self):
         """
         return self.prefix()
 
-    def _has_nontrivial_profile(self):
+    def _has_nontrivial_profile(self) -> bool:
         r"""
         Return ``True`` if the profile function for this algebra seems to be that
         for a proper sub-Hopf algebra of the Steenrod algebra.
@@ -744,7 +744,7 @@ def _has_nontrivial_profile(self):
                   or (profile[1] and min(profile[1]) == 1)))
                 or (trunc < Infinity))
 
-    def _repr_(self):
+    def _repr_(self) -> str:
         r"""
         Printed representation of the Steenrod algebra.
 
@@ -799,7 +799,7 @@ def abridge_list(l):
             return "sub-Hopf algebra of %smod %d Steenrod algebra, %s basis, profile function %s" % (genprefix, self.prime(), self._basis_name, pro_str)
         return "%smod %d Steenrod algebra, %s basis" % (genprefix, self.prime(), self._basis_name)
 
-    def _latex_(self):
+    def _latex_(self) -> str:
         r"""
         LaTeX representation of the Steenrod algebra.
 
@@ -813,7 +813,7 @@ def _latex_(self):
         """
         return "\\mathcal{A}_{%s}" % self.prime()
 
-    def _repr_term(self, t):
+    def _repr_term(self, t) -> str:
         r"""
         String representation of the monomial specified by the tuple ``t``.
 
@@ -2405,17 +2405,15 @@ def Q_exp(self, *nums):
         if self.basis_name() != 'milnor':
             return self(SteenrodAlgebra(p=self.prime(),
                                         generic=self._generic).Q_exp(*nums))
-        while nums[-1] == 0:
-            nums = nums[:-1]
+
+        lnums = list(nums)
+        while lnums[-1] == 0:
+            lnums.pop()
+
         if not self._generic:
-            return self.P(*nums)
+            return self.P(*lnums)
 
-        mono = ()
-        index = 0
-        for e in nums:
-            if e == 1:
-                mono = mono + (index,)
-            index += 1
+        mono = (index for index, e in enumerate(lnums) if e == 1)
         return self.Q(*mono)
 
     def Q(self, *nums):