Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(80309)

#12750: datetime.strftime('%s') should respect tzinfo

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 11 months ago by darius
Modified:
5 years, 1 month ago
Reviewers:
alexander.belopolsky, mumino, 4kir4.1i
CC:
tim.peters, sasha, haypo, rbcollins, r.david.murray, santa4nt, akira, bignose, darius_dons.net.au, mumino, shanmbic, p.ganssle_gmail.com, awilliam_redhat.com
Visibility:
Public.

Patch Set 1 #

Total comments: 1

Patch Set 2 #

Patch Set 3 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
Doc/library/datetime.rst View 1 2 2 chunks +9 lines, -1 line 2 comments Download
Lib/datetime.py View 1 2 2 chunks +6 lines, -1 line 0 comments Download
Lib/test/datetimetester.py View 1 2 2 chunks +41 lines, -0 lines 0 comments Download
Modules/_datetimemodule.c View 1 2 4 chunks +46 lines, -0 lines 0 comments Download

Messages

Total messages: 3
sasha
http://bugs.python.org/review/12750/diff/6015/Lib/datetime.py File Lib/datetime.py (right): http://bugs.python.org/review/12750/diff/6015/Lib/datetime.py#newcode224 Lib/datetime.py:224: sreplace = '%d' % object.timestamp() I am not sure ...
6 years, 11 months ago #1
mumino
On 2012/09/26 00:30:34, sasha wrote: > http://bugs.python.org/review/12750/diff/6015/Lib/datetime.py > File Lib/datetime.py (right): > > http://bugs.python.org/review/12750/diff/6015/Lib/datetime.py#newcode224 > ...
6 years, 10 months ago #2
akira
5 years, 1 month ago #3
http://bugs.python.org/review/12750/diff/12306/Doc/library/datetime.rst
File Doc/library/datetime.rst (right):

http://bugs.python.org/review/12750/diff/12306/Doc/library/datetime.rst#newco...
Doc/library/datetime.rst:1899: |           | the Epoch,                     |
(1988-08-16 21:30:00   |       |
Why not use alphabetical order?

http://bugs.python.org/review/12750/diff/12306/Doc/library/datetime.rst#newco...
Doc/library/datetime.rst:1973: ``%s`` format code support is added for only
:meth:`strftime`. In previous
Is it mentioned somewhere that dt.strftime('%s') uses local timezone for naive
dt objects?
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+