I think so, but it might not hurt to have it. Best, Travis
On Thursday, February 22, 2024 at 9:54:32 AM UTC+9 Nils Bruin wrote: > I noticed the following cython code > > if S is long: > return sage.rings.integer.long_to_Z() > elif S is int: > return sage.rings.integer.int_to_Z() > > > https://github.com/sagemath/sage/blob/30fecca1981087a88eb8db2cf05e18edbb50d16f/src/sage/rings/integer_ring.pyx#L589C1-L593C1 > > However, in cython with python3 we now have: > > sage: cython(""" > ....: def tst(): > ....: return int is long > ....: """) > sage: tst() > True > > so I think the `elif` can be deleted. Is that correct? > -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/sage-devel/e362d62d-975c-4654-8b19-db8c88778febn%40googlegroups.com.