Author ncoghlan
Recipients docs@python, ncoghlan, r.david.murray, robert_smallshire, serhiy.storchaka
Date 2018-03-10.09:03:26
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1520672607.02.0.467229070634.issue26701@psf.upfronthosting.co.za>
In-reply-to
Content
I think __trunc__ is special here, as it's called by a built-in type constructor, whereas __floor__ and __ceil__ really are specific to their respective math module functions. That said, I also wouldn't be opposed to listing all 4 methods together (or else listing __floor__ and __ceil__ with the just added entry for __trunc__).

I also filed https://bugs.python.org/issue33039 to cover an anomaly Eric found after I suggested adding __index__ to the list of methods tried, which is that int() and math.trunc() *don't* call __index__ implicitly: the type has to set "__int__ = __index__" and "__trunc__ = __index__" to get them to work.

I suspect we just didn't notice when PEP 357 was implemented because the developers that wanted __index__ were adding it to types that *already* defined __int__ and __trunc__.
History
Date User Action Args
2018-03-10 09:03:27ncoghlansetrecipients: + ncoghlan, r.david.murray, docs@python, serhiy.storchaka, robert_smallshire
2018-03-10 09:03:27ncoghlansetmessageid: <1520672607.02.0.467229070634.issue26701@psf.upfronthosting.co.za>
2018-03-10 09:03:27ncoghlanlinkissue26701 messages
2018-03-10 09:03:26ncoghlancreate