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()
source.backup(target
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
messages: 3
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
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
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 instal
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)) = 000
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_MIN(SOMAXCONN,
128
Peter McCormick added the comment:
The smallest of typo fixes.
--
Added file: http://bugs.python.org/file39068/pdm-iofile_typo-v1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20175
Changes by Peter McCormick pe...@pdmccormick.com:
--
nosy: +pdmccormick
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20175
___
___
Python-bugs
Changes by Peter McCormick pe...@pdmccormick.com:
--
nosy: +pdmccormick
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23935
___
___
Python-bugs
Peter McCormick added the comment:
I am working on revising the Argument Clinic definitions for socketmodule.c.
--
nosy: +pdmccormick
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue20174
Peter McCormick added the comment:
Removed unrelated doc changes.
--
Added file: http://bugs.python.org/file38909/issue_10933-2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10933
Changes by Peter McCormick pe...@pdmccormick.com:
--
nosy: +pdmccormick
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10933
___
___
Python-bugs
14 matches
Mail list logo
