Alexander Belopolsky <alexander.belopol...@gmail.com> added the comment:

New patch, issue2706a.diff, implements timedelta % timedelta and addresses  
Mark's code review comments.


With respect to Mark's
"""
One other thought:  with this division operation added, it might be nice to add 
constants like td_hour, td_minute, etc. to the module.  Then the perennial 'how 
can I convert my timedelta x to minutes' questions could be answered with 'just 
do x/td_minute'.  I would personally find x/td_second to be a more obvious and 
natural way to find the total number of seconds in a timedelta than 
x.total_seconds.  I also quite like the idea of being able to create a 2.5-hour 
timedelta with something like

2*td_hour + 30*td_minute

On the other hand, maybe such constants would just be added clutter, since it's 
easy to create them when needed.
"""

I dislike this proposal for the same reason as Mark likes it: 2*td_hour + 
30*td_minute == timedelta(hours=2, minutes=30) is a violation of TOOWTDI.

----------
Added file: http://bugs.python.org/file17003/issue2706a.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue2706>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to