[issue35707] time.sleep() should support objects with __float__

Jeroen Demeyer report at bugs.python.org
Mon Jan 28 13:14:58 EST 2019


Jeroen Demeyer <J.Demeyer at UGent.be> added the comment:

> I'm also mildly concerned by how duplicative the code becomes post-patch.

I know, that's why I added that comment on GitHub.

> perhaps just implement _PyTime_ObjectToTime_t as a wrapper for _PyTime_ObjectToDenominator

Sure, but will that increase the chances of PR 11636 being accepted? Unless a core developer who is willing to merge that PR asks me that, I'd rather not add extra complications to that PR. (to be clear: I mean no offense, it's just that getting a CPython PR accepted is hard)

----------

_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue35707>
_______________________________________


More information about the Python-bugs-list mailing list