[issue21555] gcmodule.c could use pytime.h

2014-05-24 Thread Geoffrey Spear
Geoffrey Spear added the comment: Per pitrou's suggestion, eliminate get_time altogether and use _PyTime_INTERVAL macro. -- Added file: http://bugs.python.org/file35340/issue21555-3.patch ___ Python tracker rep...@bugs.python.org

[issue21555] gcmodule.c could use pytime.h

2014-05-24 Thread Roundup Robot
Roundup Robot added the comment: New changeset d151ee749f5a by Antoine Pitrou in branch 'default': Issue #21555: simplify code in gcmodule.c by using the pytime.h functions instead of trying to call time.time() via the C API. http://hg.python.org/cpython/rev/d151ee749f5a -- nosy:

[issue21555] gcmodule.c could use pytime.h

2014-05-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: Patch committed. Thank you for contributing, Geoffrey! -- resolution: - fixed stage: needs patch - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21555

[issue21555] gcmodule.c could use pytime.h

2014-05-23 Thread Geoffrey Spear
Changes by Geoffrey Spear geoffsp...@gmail.com: -- nosy: +geoffreyspear ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21555 ___ ___

[issue21555] gcmodule.c could use pytime.h

2014-05-23 Thread Geoffrey Spear
Geoffrey Spear added the comment: Attached patch replaces call to Python time.time() with _PyTime_gettimeofday() call. -- keywords: +patch Added file: http://bugs.python.org/file35324/issue21555.patch ___ Python tracker rep...@bugs.python.org

[issue21555] gcmodule.c could use pytime.h

2014-05-23 Thread Geoffrey Spear
Geoffrey Spear added the comment: revised patch; thanks to berkerpeksag's code review. -- Added file: http://bugs.python.org/file35325/issue21555-2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21555

[issue21555] gcmodule.c could use pytime.h

2014-05-22 Thread Antoine Pitrou
New submission from Antoine Pitrou: gcmodule.c is able to displayed elapsed time of garbage collection runs. It currently does it by poking inside the time module to call time.time() and convert it to a C double. It could instead use the new pytime.h facilities. -- components:

[issue21555] gcmodule.c could use pytime.h

2014-05-22 Thread Sunny K
Changes by Sunny K sunfin...@gmail.com: -- nosy: +sunfinite ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21555 ___ ___ Python-bugs-list mailing

[issue21555] gcmodule.c could use pytime.h

2014-05-22 Thread Maciej Szulik
Changes by Maciej Szulik solt...@gmail.com: -- nosy: +maciej.szulik ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21555 ___ ___ Python-bugs-list

[issue21555] gcmodule.c could use pytime.h

2014-05-22 Thread Nikolaus Rath
Changes by Nikolaus Rath nikol...@rath.org: -- nosy: +nikratio ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue21555 ___ ___ Python-bugs-list