Dino Viehland added the comment:
I ran into a similar issue and believe I have a simpler repro. I think the
issue here might be that when you take a stack overflow inside of a
sys.settrace handler that you can then later violate the stack depth. The
attached test.py has no dependencies and
New submission from Dino Viehland :
x = bytearray(b'abc')
y = memoryview(x)
del y[0:1]
This crashes CPython 3.1.1 and 2.7.
--
components: Interpreter Core
messages: 115333
nosy: dino.viehland
priority: normal
severity: normal
status: open
title: Del on memoryview crashes CP
New submission from Dino Viehland :
subprocess isn't very friendly to implementations with different GCs then
CPython and in general relies on behavior that's not going to be very stable.
I noticed the following issues but was able to work around all of them.
First Popen is a f
New submission from Dino Viehland :
This adds new test cases for bugs reported by Thomas Heller against IronPython
for ctypes and winreg:
ctypes: the variant bool type isn't supported
winreg: errno is not correctly set when QueryValue fails
--
assignee: theller
compo
Changes by Dino Viehland :
--
type: -> behavior
versions: +Python 3.2
Added file: http://bugs.python.org/file16132/patch30.diff
___
Python tracker
<http://bugs.python.org/iss
Dino Viehland added the comment:
The problem with assertRaises is I need to actually check the errno on the
exception and assertRaises doesn't let me do that.
--
___
Python tracker
<http://bugs.python.org/i
Dino Viehland added the comment:
I'll switch to the context manager form in the 3.x branch - it looks like
that's not available in the 2.x branch.
--
___
Python tracker
<http://bugs.python.
Changes by Dino Viehland :
Added file: http://bugs.python.org/file16133/patch30.diff
___
Python tracker
<http://bugs.python.org/issue7855>
___
___
Python-bugs-list mailin
Dino Viehland added the comment:
Ezio: I'm going to stick w/ assertEqual here as Michael points out these are
different. In particular I want to make sure that you can assign any true
value to the ctypes field but when you read it back you get True or False
regardless of the value ass
Changes by Dino Viehland :
Added file: http://bugs.python.org/file16134/patch.diff
___
Python tracker
<http://bugs.python.org/issue7855>
___
___
Python-bugs-list mailin
Changes by Dino Viehland :
Added file: http://bugs.python.org/file16135/patch30.diff
___
Python tracker
<http://bugs.python.org/issue7855>
___
___
Python-bugs-list mailin
Dino Viehland added the comment:
Ahh, so it is in 2.7, apparently I was in a 2.6 enlistment. I've updated the
2.x patch to use the context manager as well.
I've also switched to using assertTrue(x is True) and assertTrue(x is False) so
that it's not checking precisely for
Dino Viehland added the comment:
Brett Cannon was going to try and look at it and commit it. If you think it
looks good you can commit it :)
I do not yet have commit privledges simply because I need to gen my SSH key and
send that off to Brett. I'm also hoping to get a little hand ho
New submission from Dino Viehland <[EMAIL PROTECTED]>:
In 2.6 but not in 3.0 RC2:
x = bytearray(b'abc')
y = x.replace(b'abc', b'bar', 0)
id(x) == id(y)
In 2.6 and in 3.0 RC2:
t = bytearray()
for i in range(256): t.append(i)
x = bytearray(b
New submission from Dino Viehland <[EMAIL PROTECTED]>:
10e667.__format__('+') returns '+1.0#INF'
vs:
10e667.__format__('') which returns 'inf'
The docs say 'inf' is right.
--
messages: 76703
nosy: DinoV
severity: normal
statu
Changes by Dino Viehland <[EMAIL PROTECTED]>:
--
components: +Interpreter Core
type: -> behavior
___
Python tracker <[EMAIL PROTECTED]>
<http://bugs.py
New submission from Dino Viehland :
The idea here is to add a new flag to the vectorcall nargs that indicates the
call is being awaited: _Py_AWAITED_CALL_MARKER. This flag will allow the
callee to know that it's being eagerly evaluated. When the call is eagerly
evaluated the calle
Dino Viehland added the comment:
Doh, sorry about that link, this one goes to a specific commit:
https://github.com/facebookincubator/cinder/blob/6863212ada4b569c15cd95c4e7a838f254c8ccfb/Python/ceval.c#L6642
I do think a new opcode is a good way to go, and that could just be emitted by
the
Dino Viehland added the comment:
New changeset 992347d7376765fe3f4fc958fb1be193ba21f6c3 by Dino Viehland
(Mickaƫl Schoentgen) in branch 'master':
bpo-26185: Fix repr() on empty ZipInfo object (#13441)
https://github.com/python/cpython/commit/992347d7376765fe3f4fc958fb1be1
Change by Dino Viehland :
--
assignee: dino.viehland
components: Extension Modules
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make _posixsubprocess PEP-384 compatible
versions: Python 3.9
___
Python tracker
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15433
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15780
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
components: Extension Modules
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make termios PEP-384 compatible
type: enhancement
versions: Python 3.9
___
Python tracker
<ht
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15438
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15785
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
assignee: dino.viehland
components: Extension Modules
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make grp module PEP-384 compatible
versions: Python 3.9
___
Python tracker
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15441
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15788
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15443
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15790
___
Python tracker
<https://bugs.python.org/issu
New submission from Dino Viehland :
Make the pwd module PEP-384 compatible
--
assignee: dino.viehland
components: Extension Modules
messages: 351495
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make pwd module PEP-384 compatible
versions: Python
New submission from Dino Viehland :
Make zlib PEP-384 compatible
--
components: Extension Modules
messages: 351501
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make zlib PEP-384 compatible
versions: Python 3.9
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15445
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15792
___
Python tracker
<https://bugs.python.org/issu
New submission from Dino Viehland :
Make random module PEP-384 compatible
--
assignee: dino.viehland
components: Extension Modules
messages: 351509
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make random module PEP-384 compatible
versions
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15450
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15798
___
Python tracker
<https://bugs.python.org/issu
New submission from Dino Viehland :
Make struct module PEP-384 compatible
--
assignee: dino.viehland
components: Extension Modules
messages: 351524
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
status: open
title: Make struct module PEP-384 compatible
versions
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15456
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15805
___
Python tracker
<https://bugs.python.org/issu
Dino Viehland added the comment:
New changeset c6bbcd258302b4b9b3d4f3c39bb5f7ff0120ec67 by Dino Viehland (Neil
Schemenauer) in branch 'master':
bpo-37725: have "make clean" remove PGO task data (#15033)
https://github.com/python/cpython/commit/c6bbcd258302b4b9b3d4
Dino Viehland added the comment:
New changeset 9669931e5e76cf4b6ae6d3d66e699b5fd6ffe931 by Dino Viehland (Jeroen
Demeyer) in branch 'master':
bpo-36971: add subsections in C API "Common Object Structures" page (#13446)
https://github.com/p
Dino Viehland added the comment:
There's several different reasons why these changes are beneficial. It'll help
make the modules more compatible with PEP 554 in the future, enable more code
re-use across Python VMs, reduce code maintenance going forward, and may also
help with p
Dino Viehland added the comment:
New changeset 88bdb9280b251d753f1b1c8d9183de0fff003622 by Dino Viehland (Serhiy
Storchaka) in branch 'master':
bpo-36781: Optimize sum() for bools. (#13074)
https://github.com/python/cpython/commit/88bdb9280b251d753f1b1c8d9183de0fff003622
-
Dino Viehland added the comment:
Sorry, it seemed like a perfectly reasonable change when I was looking at the
PR!
--
___
Python tracker
<https://bugs.python.org/issue36
New submission from Dino Viehland :
Remove various static state from the ast module and make it PEP-384 compatible.
This will help make it be more compatible w/ subinterpreters and make it
re-usable by 3rd party implementations.
--
assignee: dino.viehland
components: Extension
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15592
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15957
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
nosy: +eric.snow
___
Python tracker
<https://bugs.python.org/issue38113>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Dino Viehland :
Remove static PyTypeObject* definitions and move static data to _selectstate to
help improve compatibility w/ subinterpreters and re-use by alternate
implementations.
--
assignee: dino.viehland
components: Extension Modules
messages: 351913
nosy
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15604
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/15971
___
Python tracker
<https://bugs.python.org/issu
Dino Viehland added the comment:
Remove statics to make more compatible with subinterpreters.
--
components: +Interpreter Core -Extension Modules
title: Make ast module PEP-384 compatible -> Remove statics from ast.c
___
Python tracker
<
Change by Dino Viehland :
--
pull_requests: +15608
pull_request: https://github.com/python/cpython/pull/15975
___
Python tracker
<https://bugs.python.org/issue38
New submission from Dino Viehland :
This makes it impossible to port certain types to the stable ABI and remove
statics from the interpreter.
--
assignee: dino.viehland
components: Interpreter Core
messages: 352167
nosy: dino.viehland, eric.snow
priority: normal
severity: normal
Change by Dino Viehland :
--
assignee: -> dino.viehland
___
Python tracker
<https://bugs.python.org/issue38138>
___
___
Python-bugs-list mailing list
Un
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15676
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/16053
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
assignee: -> dino.viehland
nosy: +dino.viehland
___
Python tracker
<https://bugs.python.org/issue38152>
___
___
Python-bugs-lis
Change by Dino Viehland :
--
nosy: +petr.viktorin
___
Python tracker
<https://bugs.python.org/issue38140>
___
___
Python-bugs-list mailing list
Unsubscribe:
Dino Viehland added the comment:
New changeset f919054e539a5c1afde1b31c9fd7a8f5b2313311 by Dino Viehland in
branch 'master':
bpo-38116: Convert select module to PEP-384 (#15971)
https://github.com/python/cpython/commit/f919054e539a5c1afde1b31c9fd7a8
Change by Dino Viehland :
--
assignee: -> dino.viehland
nosy: +dino.viehland
___
Python tracker
<https://bugs.python.org/issue38176>
___
___
Python-bugs-lis
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +15768
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/16158
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
stage: patch review -> resolved
___
Python tracker
<https://bugs.python.org/issue38176>
___
___
Python-bugs-list mailing list
Un
Change by Dino Viehland :
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue38176>
___
___
Python-bugs-list
Dino Viehland added the comment:
New changeset 3368f3c6ae4140a0883e19350e672fd09c9db616 by Dino Viehland (Eddie
Elizondo) in branch 'master':
bpo-38140: Make dict and weakref offsets opaque for C heap types (#16076)
https://github.com/python/cpyt
Change by Dino Viehland :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Dino Viehland added the comment:
@BTaskaya Seems done, I'll go ahead and close it
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.
Dino Viehland added the comment:
Adding the getter/setters seems perfectly reasonable to me, and I agree they
should be underscore prefixed as well.
--
___
Python tracker
<https://bugs.python.org/issue38
Dino Viehland added the comment:
Setup a micro-benchmark, foo.c:
#define PY_SSIZE_T_CLEAN
#include
#include
#include
int
main(int argc, char *argv[])
{
wchar_t *program = Py_DecodeLocale(argv[0], NULL);
if (program == NULL) {
fprintf(stderr, "Fatal error: cannot d
Dino Viehland added the comment:
Looking!
--
___
Python tracker
<https://bugs.python.org/issue43636>
___
___
Python-bugs-list mailing list
Unsubscribe:
Dino Viehland added the comment:
It's probably worth having an assert that the version tag is valid, that'll
make it easier to see what's going wrong in the cache hit case. We should have
the version tag being 0 now when it's not valid. I may not be able to debug it
Dino Viehland added the comment:
And it looks like we have an entry with a 0 version, but with a valid name:
(type_cache_entry) $3 = {
version = 0
name = 0x000100ec44f0
value = NULL
}
--
___
Python tracker
<https://bugs.python.
Dino Viehland added the comment:
I think the issue here is that in assign_version_tag there's this code:
if (type->tp_version_tag == 0) {
// Wrap-around or just starting Python - clear the whole cache
type_cache_clear(cache, 1);
return 1;
}
the ret
Dino Viehland added the comment:
@vstinner - The fix doesn't change the behavior of _PyType_Lookup and instead
just fixes a previous unrelated bug. The condition will typically ever be hit
once (at startup) as it's very unlikely that versions will wrap, so there
should be no p
Dino Viehland added the comment:
New changeset af8c8caaf5e07c02202d736a31f6a2f7e27819b8 by Weipeng Hong in
branch 'main':
bpo-30533:Add function inspect.getmembers_static that does not call properties
or dynamic properties. (#20911)
https://github.com/python/cpyt
Change by Dino Viehland :
--
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.11 -Python 3.9
___
Python tracker
<https://bugs.python.org/i
Dino Viehland added the comment:
New changeset c2bb29ce9ae4adb6a8123285ad3585907cd4cc73 by Weipeng Hong in
branch 'main':
bpo-30533: Add docs for `inspect.getmembers_static` (#29874)
https://github.com/python/cpython/commit/c2bb29ce9ae4adb6a8123285ad3585
Change by Dino Viehland :
--
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue30533>
___
___
Pyth
New submission from Dino Viehland :
I'm trying to create a custom module type for a custom loader where the
returned modules are immutable. But I'm running into an issue where the
immutable module type can't be used as a module for a package. That's because
the im
Change by Dino Viehland :
--
nosy: +brett.cannon
___
Python tracker
<https://bugs.python.org/issue39336>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +17406
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/18006
___
Python tracker
<https://bugs.python.org/issu
Dino Viehland added the comment:
I think the warning shouldn't be too bad. It looks like ImportWarnings are
filtered by default already, and the extra overhead of raising a warning in
this case probably is nothing compared to the actual work in loading the m
Change by Dino Viehland :
--
nosy: +eelizondo -dino.viehland
___
Python tracker
<https://bugs.python.org/issue38076>
___
___
Python-bugs-list mailing list
Unsub
Dino Viehland added the comment:
It seems problematic that_PyInterpreterState_ClearModules runs before all
instances from a module have been cleared. If PyState_FindModule is no longer
able to return module state then there's no way for a module to reliably work
at shutdown other
Dino Viehland added the comment:
This is a relatively simple repro of the underlying problem:
import _struct
s = _struct.Struct('i')
class C:
def __del__(self):
s.pack(42, 100)
_struct.x = C()
It's a little bit different in that it is actually causing the module
Dino Viehland added the comment:
And here's a variation which doesn't involve any instances from the module:
import _struct
class C:
def __init__(self):
self.pack = _struct.pack
def __del__(self):
self.pack('I',
Change by Dino Viehland :
--
pull_requests: +17437
stage: resolved -> patch review
pull_request: https://github.com/python/cpython/pull/18038
___
Python tracker
<https://bugs.python.org/issu
Dino Viehland added the comment:
https://github.com/python/cpython/pull/18038 is a partial fix for this. I
think it fixes the crash at shutdown, although I'm still seeing a hang on
master on Linux which is different then earlier versions of Python. I seem to
have a really bogus
Dino Viehland added the comment:
With either fix, or with both, on Linux I still see this hang at shutdown.
Victor mentioned the fact that he had to hit Ctrl-C on Linux to see this, and I
have to do the same thing. Then with the fixes in place the original test case
still hangs on shutdown
Dino Viehland added the comment:
New changeset 9b6fec46513006d7b06fcb645cca6e4f5bf7c7b8 by Dino Viehland in
branch 'master':
bpo-39336: Allow packages to not let their child modules be set on them (#18006)
https://github.com/python/cpython/commit/9b6fec46513006d7b06fcb645cca6e
Change by Dino Viehland :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Dino Viehland added the comment:
One more data point: Backporting this change to Python 3.6 (I just happened to
have it applied there already, so I haven't tried it on 3.7 or 3.8) has no
crash and no hangs in multiprocessing on Linux. So something definitely
changed in multiproessing
Dino Viehland added the comment:
The curious thing about this is other tests in CircularImportTests are
importing packages from test.test_import.data in the exact same way.
--
___
Python tracker
<https://bugs.python.org/issue39
Dino Viehland added the comment:
Ahh, that's probably it Brett, I didn't know that was there, thanks!
--
___
Python tracker
<https://bugs.python.o
Change by Dino Viehland :
--
pull_requests: +17590
pull_request: https://github.com/python/cpython/pull/18211
___
Python tracker
<https://bugs.python.org/issue39
Dino Viehland added the comment:
I've added the files to the makefile and AMD64 Fedora Stable Clang Installed
3.x was passing.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python t
Change by Dino Viehland :
--
pull_requests: +17621
pull_request: https://github.com/python/cpython/pull/18241
___
Python tracker
<https://bugs.python.org/issue39
Dino Viehland added the comment:
Nope, thank you for pointing that out. I've updated them now with PR 18241
--
___
Python tracker
<https://bugs.python.org/is
Dino Viehland added the comment:
New changeset 0cd5bff6b7da3118d0c5a88fc2b80f80eb7c3059 by Dino Viehland in
branch 'master':
bpo-39459: include missing test files in windows installer
https://github.com/python/cpython/commit/0cd5bff6b7da3118d0c5a88fc2b80f
Dino Viehland added the comment:
I guess the update to lib.pyproj probably just makes the files show up when
opening the solution in Visual Studio then.
--
___
Python tracker
<https://bugs.python.org/issue39
New submission from Dino Viehland :
The fix for bpo-17636 added support for falling back to sys.modules when a
module isn't directly present on the module. But mock doesn't have the same
behavior - it'll try the import, and then try to get the value off the object.
If it
Change by Dino Viehland :
--
keywords: +patch
pull_requests: +17721
stage: needs patch -> patch review
pull_request: https://github.com/python/cpython/pull/18347
___
Python tracker
<https://bugs.python.org/issu
Change by Dino Viehland :
--
pull_requests: +17722
pull_request: https://github.com/python/cpython/pull/18347
___
Python tracker
<https://bugs.python.org/issue17
Dino Viehland added the comment:
It's related to bpo-39336. If you have an immutable package which doesn't
allow it's children to be published on it when following the chain of packages
it ends up not arriving at the final module.
But you can also hit it if you end up
Dino Viehland added the comment:
My actual scenario involves a custom module loader where the modules are
published are completely immutable (it ends up publishing an object which isn't
a subtype of module). It can still have normal Python modules as a child which
aren't immutabl
Dino Viehland added the comment:
Sorry, publish may not necessarily be the best term. When you do "from foo
import bar" and foo is a package Python will set the bar module onto the foo
module. That leads to the common mistake of doing "import foo" and then
"foo.ba
Dino Viehland added the comment:
I like that idea, let me see if I can find a way to do that. This is a little
bit different in that it's implicitly trying to find a module, and supports
dotting through non-packages as well, but maybe there's a way to leverage
importlib and stil
Dino Viehland added the comment:
New changeset 52268941f37e3e27bd01792b081877ec3bc9ce12 by Matthias Braun in
branch 'master':
bpo-26067: Do not fail test_shutil / chown when gid/uid cannot be resolved
(#19032)
https://github.com/python/cpython/commit/52268941f37e3e27bd01792b081877
Dino Viehland added the comment:
I think there's other cases of performance related features being hidden under
an ifdef. Computed gotos show up that way, although probably more because it's
a compiler extension that's not supported everywhere. Pymalloc is also very
sim
1 - 100 of 134 matches
Mail list logo