Peter McCormick added the comment:
Updated sample script, thanks to @aeros for catching the omission:
```
import sqlite3
target = sqlite3.connect(':memory:')
source = sqlite3.connect(':memory:')
source.close()
New submission from Peter McCormick :
Attempting to backup a closed database will trigger segfault:
```
target = sqlite.connect(':memory:')
source = sqlite.connect(":memory:")
source.close()
source.backup(target)
```
--
files: fix.patch
keywords: patch
m
Peter McCormick added the comment:
I would like to propose the following patch[0] which generalizes the
`PyDTrace_*` definitions to a more generic `PyProbe` structure, in anticipation
of including LTTng support side-by-side with the existing DTrace support.
A couple of argument types in
Peter McCormick added the comment:
Hi Łukasz, thank you for the feedback!
> "PyTrace" is already a name in use for a different purpose. I understand the
> itch to make the name more "right" but I am in general not a fan of renaming
> "PyDTrace" to anyth
Peter McCormick added the comment:
A few suggestions:
* Disallow `--with-lttngust` on anything other than Linux (on macOS
`configure` dies due to differences in acceptable `mktemp` arguments if you
even attempt it)
* Rename `cpython_inst.h` to `pytrace.h` and rename `pylttngust_probes.h` to
Peter McCormick added the comment:
Fork at
<https://github.com/pdmccormick/cpython/tree/bpo-28909-adding-lttng-ust> with
Francis' original patch, plus a revert for the DTrace provider name change.
Here are instructions I used to try this out:
## LTTng installation
Full i
Peter McCormick added the comment:
The smallest of typo fixes.
--
Added file: http://bugs.python.org/file39068/pdm-iofile_typo-v1.patch
___
Python tracker
<http://bugs.python.org/issue20
Changes by Peter McCormick :
--
nosy: +pdmccormick
___
Python tracker
<http://bugs.python.org/issue20175>
___
___
Python-bugs-list mailing list
Unsubscribe:
Peter McCormick added the comment:
I missed the fact that Larry's patch obviates the need for the `builtins.`
prefix, shortening the Argument Clinic parameter specification into:
backlog: int(py_default="min(SOMAXCONN, 128)", c_default="Py
Peter McCormick added the comment:
This definitely works for the _socket.listen use case!
In terms of generating such a signature using Argument Clinic, currently this
is required:
backlog: int(py_default="builtins.min(SOMAXCONN, 128)",
c_default="Py_MIN(SOMAXCONN, 128
Changes by Peter McCormick :
--
nosy: +pdmccormick
___
Python tracker
<http://bugs.python.org/issue23935>
___
___
Python-bugs-list mailing list
Unsubscribe:
Peter McCormick added the comment:
I am working on revising the Argument Clinic definitions for socketmodule.c.
--
nosy: +pdmccormick
___
Python tracker
<http://bugs.python.org/issue20
Peter McCormick added the comment:
Removed unrelated doc changes.
--
Added file: http://bugs.python.org/file38909/issue_10933-2.patch
___
Python tracker
<http://bugs.python.org/issue10
Changes by Peter McCormick :
--
nosy: +pdmccormick
___
Python tracker
<http://bugs.python.org/issue10933>
___
___
Python-bugs-list mailing list
Unsubscribe:
14 matches
Mail list logo
