Changes by Andrew Svetlov andrew.svet...@gmail.com:
--
nosy: +asvetlov
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9622
___
___
Python-bugs-list
Changes by Terry J. Reedy tjre...@udel.edu:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9622
___
___
Python-bugs-list
Antoine Pitrou pit...@free.fr added the comment:
Adding new global objects looks a bit complicated. Why not simply update all
thread-local objects when the global flag is set?
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
Kristján Valur Jónsson krist...@ccpgames.com added the comment:
Good question. Is that simple to do? I didn't think to look if there were a
central list of all python TLS states.
--
___
Python tracker rep...@bugs.python.org
Antoine Pitrou pit...@free.fr added the comment:
Good question. Is that simple to do? I didn't think to look if there
were a central list of all python TLS states.
There's a linked list from the current interpreter state, which in turn
can be got from the current thread state (see
Kristján Valur Jónsson krist...@ccpgames.com added the comment:
Sounds like a much simpler change. I'll try that out instead.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9622
___
Kristján Valur Jónsson krist...@ccpgames.com added the comment:
Here is a new, much simpler patch, which simply sets the flags on all the
threads (accessable as a linked list of PyThreadState objects) as suggested by
Antoine.
(Note that neither of the patches has the necessary enhancements to
Jon Parise j...@indelible.org added the comment:
There are a few places in the patch where you call the global version
immediately followed by the local version. For example:
+PyEval_SetGlobalTrace(NULL, NULL);
PyEval_SetTrace(NULL, NULL);
Isn't the local call now redundant
Kristján Valur Jónsson krist...@ccpgames.com added the comment:
Indeed it is. This is a remnant of the previous method. I will fix it, and
add documentation. I also think I'll leave the change to _lsprof out of this
so that this change can stand on its own, irrespective of the profiling
Kristján Valur Jónsson krist...@ccpgames.com added the comment:
I just realized that this is probably a redundant change.
We have C apis to get all the Thread states in an interpreter state (I didn't
even know there was such a thing as multiple interpreter states, but there!)
This is the
Kristján Valur Jónsson krist...@ccpgames.com added the comment:
Setting this to invalid, since there already are APIs to do this, at least from
.py code.
--
resolution: - invalid
___
Python tracker rep...@bugs.python.org
New submission from Kristján Valur Jónsson krist...@ccpgames.com:
issue 9609 updates _lsprof.c to be multi-stack aware. This allows
cProfile.Profile() objects to be shared by many threads and provide meaningfull
results.
This update makes it more convenient to profile running,
12 matches
Mail list logo