[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-18 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-18 Thread Roundup Robot
Roundup Robot added the comment: New changeset 52931189d7ee by Serhiy Storchaka in branch 'default': Issue #27342: Replaced some Py_XDECREFs with Py_DECREFs. https://hg.python.org/cpython/rev/52931189d7ee -- nosy: +python-dev ___ Python tracker

[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-17 Thread Raymond Hettinger
Raymond Hettinger added the comment: v2 looks correct. -- nosy: +rhettinger ___ Python tracker ___ ___

[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-17 Thread Xiang Zhang
Xiang Zhang added the comment: Thanks for review. Eliminate the wrongs ones. -- Added file: http://bugs.python.org/file43440/cleanup_some_xdecrefs_v2.patch ___ Python tracker

[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Mostly good except longrangeiter_dealloc(). -- components: +Interpreter Core nosy: +mark.dickinson stage: -> patch review type: -> enhancement versions: +Python 3.6 ___ Python tracker

[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-17 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- nosy: +serhiy.storchaka ___ Python tracker ___

[issue27342] Clean up some Py_XDECREFs in rangeobject.c and bltinmodule.c

2016-06-17 Thread Xiang Zhang
New submission from Xiang Zhang: Some Py_XDECREFs in rangeobject.c and bltinmodule.c are not necessary. Py_DECREF is enough. Clean up them. -- files: cleanup_some_xdecrefs.patch keywords: patch messages: 268711 nosy: xiang.zhang priority: normal severity: normal status: open title: