This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Several Py_XDE/INCREFs in typeobject.c are not necessary
Type: enhancement Stage: resolved
Components: Interpreter Core Versions: Python 3.6
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: serhiy.storchaka Nosy List: python-dev, serhiy.storchaka, vstinner, xiang.zhang
Priority: normal Keywords: patch

Created on 2016-05-25 06:12 by xiang.zhang, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
Py_XDECREF_in_typeobject.diff xiang.zhang, 2016-05-25 06:12 review
Messages (6)
msg266307 - (view) Author: Xiang Zhang (xiang.zhang) * (Python committer) Date: 2016-05-25 06:12
Several Py_XDECREFs are not necessary since the pointers they handle cannot be NULL.
msg266310 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2016-05-25 06:39
LGTM.
msg266342 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2016-05-25 13:03
LGTM too.

Serhiy, do you want to push the patch?
msg266345 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2016-05-25 13:15
New changeset 3b4c3dda679a by Serhiy Storchaka in branch 'default':
Issue #27118: Clean up Py_XINCREF/Py_XDECREF in typeobject.c.
https://hg.python.org/cpython/rev/3b4c3dda679a
msg266346 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2016-05-25 13:16
Done. Thank you for your contribution Xiang Zhang.
msg266355 - (view) Author: Xiang Zhang (xiang.zhang) * (Python committer) Date: 2016-05-25 13:48
It's my pleasure and thanks for your time reviewing this thread too.
History
Date User Action Args
2022-04-11 14:58:31adminsetgithub: 71305
2016-05-25 13:48:29xiang.zhangsetmessages: + msg266355
2016-05-25 13:16:35serhiy.storchakasetstatus: open -> closed
resolution: fixed
messages: + msg266346

stage: commit review -> resolved
2016-05-25 13:15:27python-devsetnosy: + python-dev
messages: + msg266345
2016-05-25 13:03:00vstinnersetnosy: + vstinner
messages: + msg266342
2016-05-25 12:43:04serhiy.storchakasetassignee: serhiy.storchaka
2016-05-25 06:39:22serhiy.storchakasettype: behavior -> enhancement
messages: + msg266310
stage: patch review -> commit review
2016-05-25 06:30:24SilentGhostsetnosy: + serhiy.storchaka

type: enhancement -> behavior
components: + Interpreter Core
stage: patch review
2016-05-25 06:14:39xiang.zhangsettitle: Several Py_XDECREFs in typeobject.c are not necessary -> Several Py_XDE/INCREFs in typeobject.c are not necessary
2016-05-25 06:13:11xiang.zhangsetversions: + Python 3.6
type: enhancement
title: Several Py_XDECREFs are not necessary -> Several Py_XDECREFs in typeobject.c are not necessary
2016-05-25 06:12:40xiang.zhangcreate