[issue27661] Add tzinfo= argument to datetime.combine

Roundup Robot added the comment: New changeset adce94a718e3 by Alexander Belopolsky in branch 'default': Closes #27661: Added tzinfo keyword argument to datetime.combine. https://hg.python.org/cpython/rev/adce94a718e3 -- nosy: +python-dev resolution: -> fixed stage: commit review -> res

[issue27661] Add tzinfo= argument to datetime.combine

Changes by Alexander Belopolsky : Added file: http://bugs.python.org/file43965/issue27661-3.diff ___ Python tracker ___ ___ Python-bugs-list m

[issue27661] Add tzinfo= argument to datetime.combine

Alexander Belopolsky added the comment: >From review comments: Lib/datetime.py:1482: def combine(cls, date, time, tzinfo=True): On 2016/08/01 12:23:12, berkerpeksag wrote: > On 2016/08/01 08:47:14, SilentGhost wrote: > > This strikes me as an odd default value, why not use a more conventional No

[issue27661] Add tzinfo= argument to datetime.combine

Alexander Belopolsky added the comment: >From review comments: Lib/datetime.py:1482: def combine(cls, date, time, tzinfo=True): On 2016/08/01 08:47:14, SilentGhost wrote: > This strikes me as an odd default value, why not use a more conventional None? This is the same default as used in the .re

[issue27661] Add tzinfo= argument to datetime.combine

Alexander Belopolsky added the comment: The second patch includes documentation changes and addresses review commments. -- Added file: http://bugs.python.org/file43964/issue27661-2.diff ___ Python tracker _

[issue27661] Add tzinfo= argument to datetime.combine

Changes by SilentGhost : -- nosy: +SilentGhost ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyt

[issue27661] Add tzinfo= argument to datetime.combine

Changes by Alexander Belopolsky : -- keywords: +patch stage: -> patch review Added file: http://bugs.python.org/file43958/issue27661.diff ___ Python tracker ___

[issue27661] Add tzinfo= argument to datetime.combine

New submission from Alexander Belopolsky: Add an optional tzinfo argument to datetime.combine() so that datetime.combine(d, t, info) returns the same object as datetime.combine(d, t).replace(tzinfo=info) but without creating an intermediate naive instance. Guido's LGTM: https://mail.pytho