add TODO about is_unit vs is_one

vbraun · Jan 20, 2025 · 5978331 · 5978331
1 parent 485a2a5
commit 5978331
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/sage/categories/unique_factorization_domains.py b/src/sage/categories/unique_factorization_domains.py
@@ -182,6 +182,11 @@ def content(X):
                 # heuristically, polynomials tend to be monic
                 X_it = reversed(X.coefficients())
                 x = next(X_it)
+                # TODO: currently, there is no member of
+                # `UniqueFactorizationDomains` with `is_unit`
+                # significantly slower than `is_one`, see
+                # :issue:`38924` - when such a domain is eventually
+                # implemented, check whether this is a bottleneck
                 if x.is_unit():
                     return None
                 for c in X_it: