Remove _Py_PyAtExit from Python.h

[nascheme]

BPO	32310
Nosy	@nascheme, @vstinner
PRs	bpo-32310: Remove _Py_PyAtExit from Python.h. #4849

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2018-09-06.14:29:10.445>
created_at = <Date 2017-12-13.20:25:35.356>
labels = ['interpreter-core', 'type-feature']
title = 'Remove _Py_PyAtExit from Python.h'
updated_at = <Date 2018-09-06.14:29:10.443>
user = 'https://github.com/nascheme'

bugs.python.org fields:

activity = <Date 2018-09-06.14:29:10.443>
actor = 'vstinner'
assignee = 'none'
closed = True
closed_date = <Date 2018-09-06.14:29:10.445>
closer = 'vstinner'
components = ['Interpreter Core']
creation = <Date 2017-12-13.20:25:35.356>
creator = 'nascheme'
dependencies = []
files = []
hgrepos = []
issue_num = 32310
keywords = ['patch']
message_count = 2.0
messages = ['308242', '324694']
nosy_count = 2.0
nosy_names = ['nascheme', 'vstinner']
pr_nums = ['4849']
priority = 'low'
resolution = 'rejected'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue32310'
versions = []

[nascheme]

_Py_PyAtExit only supports on callback function. Its sole use it to be used by atexit. IMHO, it should be removed from Python.h to prevent misuse.

[vstinner]

It's unclear if this change is going to break anything. In case of doubt, I chose to reject the change.

Note: I'm working on a project to rework and cleanup the C API, see:
https://pythoncapi.readthedocs.io/
https://pythoncapi.readthedocs.io/split_include.html