[issue31827] Remove os.stat_float_times()

2017-10-24 Thread STINNER Victor
STINNER Victor added the comment: os.stat_float_times() has been removed, I close the issue. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue31827] Remove os.stat_float_times()

2017-10-24 Thread STINNER Victor
STINNER Victor added the comment: Copy of my comment on the PR: "So, I tried to remove the backward compatibility layer: I modified stat_result[ST_MTIME] to return float rather than int. Problem: it broke test_logging, the code deciding if a log file should be

[issue31827] Remove os.stat_float_times()

2017-10-24 Thread STINNER Victor
STINNER Victor added the comment: New changeset 01b5aab7bfb11ee5476ef52d24495598cbe7c99a by Victor Stinner in branch 'master': bpo-31827: Remove os.stat_float_times() (GH-4061) https://github.com/python/cpython/commit/01b5aab7bfb11ee5476ef52d24495598cbe7c99a

[issue31827] Remove os.stat_float_times()

2017-10-21 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: stat_result is a named 10-tuple, containing several additional attributes. The last three items are st_atime, st_mtime and st_ctime as integers. Accessing them by name returns floats. Isn't a time to make them floats when access

[issue31827] Remove os.stat_float_times()

2017-10-20 Thread STINNER Victor
STINNER Victor added the comment: Attached PR 4061 removes os.stat_float_times(). -- components: +Library (Lib) ___ Python tracker

[issue31827] Remove os.stat_float_times()

2017-10-20 Thread STINNER Victor
Change by STINNER Victor : -- keywords: +patch pull_requests: +4030 stage: -> patch review ___ Python tracker ___

[issue31827] Remove os.stat_float_times()

2017-10-20 Thread STINNER Victor
New submission from STINNER Victor : os.stat_float_times() was introduced in Python 2.3 to get file modification times with sub-second resolution. The default remains to get time as seconds (integer). See commit f607bdaa77475ec8c94614414dc2cecf8fd1ca0a. The function