[issue4934] tp_del and tp_version_tag undocumented

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Both tp_del and tp_version_tag are for internal use.  Besides, tp_del is 
obsolete as I mentioned above.  So I'm going to close the issue.

--
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Antoine Pitrou]

Antoine Pitrou  added the comment:

The reason tp_del has remained undocumented is that it's now obsolete.  You 
should use tp_finalize instead:
https://docs.python.org/3/c-api/typeobj.html#c.PyTypeObject.tp_finalize

tp_finalize is roughly the C equivalent of __del__ (tp_del was something else, 
despite the name).

--
versions: +Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[book book]

book book  added the comment:

tp_del still undocumented.

In my opinion, tp_del corresponds to the __del__() method of classes, because 
there are no other variable correspondes to it, but methods using for 
creating(tp_new) and initializing(tp_init) an object are not.

Although __del__ method have some issue according to the offical document, but 
it is exist, so there must have a pointer to the function corresponding to it.

--
nosy: +book book
versions: +Python 3.6 -Python 2.6, Python 2.7, Python 3.2, Python 3.3, Python 
3.4

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Alex Leach]

Alex Leach added the comment:

I've just ran into tp_version_tag, when running the boost python testsuite and 
wondered what it was... Since upgrading to GCC 4.8, I've started to get a lot 
more warnings with Python extensions, e.g.:-

boost/python/opaque_pointer_converter.hpp:122:14: warning: missing initializer 
for member ‘_typeobject::tp_version_tag’ [-Wmissing-field-initializers]

In this instance the testsuite was made to compile with the '-Wextra' flag. The 
fix was pretty simple; add another zero to the opaque_pointer_convert struct.

I have used the following preprocessor macro to test whether or not to do this. 
Would this be a good way to test for the addition?

#if PY_VERSION_HEX = 0x0206
  0,/* tp_version_tag */
#endif

Cheers,
Alex

--
nosy: +Alex.Leach

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Antoine Pitrou]

Antoine Pitrou added the comment:

 tp_cache and tp_weaklist are also for internal use only, but are
 documented.

Ok, so I guess tp_version_tag and tp_del should also be documented as for 
internal use only.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Ronald Oussoren]

Ronald Oussoren added the comment:

tp_cache and tp_weaklist are also for internal use only, but are documented.

One reason for documenting them is that users will run into them when running 
with a high enough warning level in GCC.

--
nosy: +ronaldoussoren

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Ezio Melotti]

Changes by Ezio Melotti ezio.melo...@gmail.com:


--
type:  - enhancement
versions: +Python 3.3, Python 3.4 -Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Martin von Gagern]

Changes by Martin von Gagern martin.vgag...@gmx.net:


--
nosy: +gagern

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Mark Lawrence]

Changes by Mark Lawrence breamore...@yahoo.co.uk:


--
assignee: georg.brandl - d...@python
nosy: +d...@python
versions: +Python 3.2 -Python 3.0

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Antoine Pitrou]

Changes by Antoine Pitrou pit...@free.fr:


--
nosy: +benjamin.peterson

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4934] tp_del and tp_version_tag undocumented

[Daniel Stutzbach]

New submission from Daniel Stutzbach dan...@stutzbachenterprises.com:

The PyTypeObject field tp_version_tag (new in 2.6) isn't documented.

tp_del isn't documented either.  I'm not sure when it was added.

--
assignee: georg.brandl
components: Documentation
messages: 79755
nosy: georg.brandl, stutzbach
severity: normal
status: open
title: tp_del and tp_version_tag undocumented
versions: Python 2.6, Python 2.7, Python 3.0, Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4934
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com