[issue5654] Add C hook in PyDict_SetItem for debuggers

STINNER Victor added the comment: No activity for 9 years, I close the issue. -- resolution: -> rejected stage: needs patch -> resolved status: open -> closed ___ Python tracker _

[issue5654] Add C hook in PyDict_SetItem for debuggers

Change by Piotr Dobrogost : -- nosy: +piotr.dobrogost ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://

[issue5654] Add C hook in PyDict_SetItem for debuggers

Martin v. Löwis added the comment: I think it's besides the point of this patch to ifdef it out. If John wants a version of Python with this enabled, he can well enough build one without our permission. So it would be useful only if it was always available, and perhaps properly integrated int

[issue5654] Add C hook in PyDict_SetItem for debuggers

STINNER Victor added the comment: > the cost in the normal case is a single pointer comparison with NULL, > something very cheap. The Linux kernel patchs dynamically the code: disabling a probe writes NOP instruction in memory to avoid the cost of the test. Probes are completly disabled by de

[issue5654] Add C hook in PyDict_SetItem for debuggers

Antoine Pitrou added the comment: I see no reason to close until benchmark results prove it decreases real-life performance. Looking at the patch, the cost in the normal case is a single pointer comparison with NULL, something very cheap. -- nosy: +pitrou

[issue5654] Add C hook in PyDict_SetItem for debuggers

Raymond Hettinger added the comment: Can this be closed? -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: htt

[issue5654] Add C hook in PyDict_SetItem for debuggers

Changes by Ezio Melotti : -- stage: -> needs patch type: -> feature request versions: +Python 3.3 -Python 2.7 ___ Python tracker ___

[issue5654] Add C hook in PyDict_SetItem for debuggers

Guido van Rossum added the comment: John, I'm adding a +0 to cancel out Raymond's -1. I've read your motivation and, like you, hope/expect that benchmarking will show this has no real impact. But I need data to decide. Once there are benchmark results I'll revise my view. A good place to star

[issue5654] Add C hook in PyDict_SetItem for debuggers

John Ehresman added the comment: My hope is that the runtime performance cost will be negligible but if it isn't, it probably shouldn't go in. The issue with not putting it in another build is that many python debugger users won't want to recompile, so I see it as being of limited use if it'

[issue5654] Add C hook in PyDict_SetItem for debuggers

Raymond Hettinger added the comment: This is a critical path in Python and it should be kept as clean as possible. If something like this goes in, it should be #ifdef'd out by default; otherwise, we have everyone paying the price for a feature that almost no one will use. Strong -1 against thi

[issue5654] Add C hook in PyDict_SetItem for debuggers

John Ehresman added the comment: Oops, the multiprocessing changes should not be in the patch -- nosy: +sdeibel ___ Python tracker ___ ___

[issue5654] Add C hook in PyDict_SetItem for debuggers

Martin v. Löwis added the comment: The patch seems to contain unrelated changes to multiprocessing. -- nosy: +loewis ___ Python tracker ___ __

[issue5654] Add C hook in PyDict_SetItem for debuggers

Changes by John Ehresman : Added file: http://bugs.python.org/file13550/testhook.py ___ Python tracker ___ ___ Python-bugs-list mailing list Un

[issue5654] Add C hook in PyDict_SetItem for debuggers

New submission from John Ehresman : I'm interested in adding support for debugger watchpoint's in the core. A watchpoint would cause the debugger to stop when a value in a namespace changes. This hook would be called when PyDict_SetItem & PyDict_DelItem is invoked. I realize that this does not