_PyTrace_init declaration
I'm getting "no prototype" warnings on _PyTrace_init, and inspection shows that this routine indeed doesn't show up in an include file. As it is used elsewhere (in sysmodule.c) shouldn't it be called PyTrace_init and have it's prototype declared somewhere? -- Jack Jansen | ++++ stop the execution of Mumia Abu-Jamal ++++ Jack.Jansen@oratrix.com | ++++ if you agree copy these lines to your sig ++++ www.oratrix.nl/~jack | ++++ see http://www.xs4all.nl/~tank/ ++++
On Wed, Jun 20, 2001 at 11:52:16PM +0200, Jack Jansen wrote:
I'm getting "no prototype" warnings on _PyTrace_init, and inspection shows that this routine indeed doesn't show up in an include file. As it is used elsewhere (in sysmodule.c) shouldn't it be called PyTrace_init and have it's prototype declared somewhere?
No, and yes. the _Py* functions are internal, but non-static (used in other
files.) They should have a prototype declared somewhere, but they shouldn't
be used outside of Python itself. It shouldn't be named 'PyTrace_init'
unless it is a supported part of the API.
--
Thomas Wouters
[Jack Jansen]
I'm getting "no prototype" warnings on _PyTrace_init, and inspection shows that this routine indeed doesn't show up in an include file. As it is used elsewhere (in sysmodule.c) shouldn't it be called PyTrace_init and have it's prototype declared somewhere?
It should indeed be declared in ceval.h (Fred?), but so long as it's part of the private API it should not lose the leading underscore.
Jack Jansen writes:
I'm getting "no prototype" warnings on _PyTrace_init, and inspection shows that this routine indeed doesn't show up in an include file. As it is used elsewhere (in sysmodule.c) shouldn't it be called PyTrace_init and have it's prototype declared somewhere?
No. I thought I had a prototype for it just above the usage. Any, I'm re-working that code this week, so you can assign this to me in the bug tracker. Thanks! -Fred -- Fred L. Drake, Jr. <fdrake at acm.org> PythonLabs at Digital Creations
participants (4)
-
Fred L. Drake, Jr. -
Jack Jansen -
Thomas Wouters -
Tim Peters