New submission from Vlad :
When dealing with a new connection, SocketServer.BaseRequestHandler.__init__
first calls the request handler (self.handle below) and then calls cleanup code
which closes the connection (self.finish below).
class BaseRequestHandler:
def __init__(self, request
New submission from Vlad :
If a class member is declared outside of a function, it's creation is
_not_ repeated with every new instance of the class. Instead, the same
member is created during the first instance and then shared by all the
subsequent instances of that class. This is
New submission from Vlad:
This issue is for Python3.3 and doesn't exist in Python3.2
Detailed description with source code can be found here:
http://stackoverflow.com/questions/15387035/second-python-execution-fails
--
components: None
messages: 184081
nosy: theDarkBrainer
pri
Vlad added the comment:
I'm trying to embed the python 3.3 engine for an app that need to run custom
scripts in python. Since the scripts might be completely different, and
sometimes user provided, I am trying to make each execution isolated and there
is not need to preserve any data be
Vlad Riscutia added the comment:
Attached doc update against tip (though I still hope my patch for configurable
allocation strategies will make it in).
This is my first doc patch so let me know if I missed something. I am basically
explaining that bit field allocation is compiler-specific
Vlad Riscutia added the comment:
I attached a patch which replaces all asserts with checks that raise
exceptions. I used my judgement in determining exception types but I might have
been off in some places. Also, this patch replaces ALL asserts. It is possible
that some of the internal
Vlad Riscutia added the comment:
Thanks for the quick review! I attached second iteration addressing feedback +
changed all occurrences of checks like "type(x) is y" to "isinstance(x, y)".
I would appreciate a second look because this patch has many small changes and
ev
Vlad Riscutia added the comment:
Thanks for the "make patchcheck" tip, I didn't know about that. I will update
the patch soon.
In the mean time, I want to point out a couple of things:
First, I'm saying "toying with the underlying buffer" because none of the
Vlad Riscutia added the comment:
I agree compiler matters for alignment but if you look at PyCField_FromDesc,
you will see the layout is pretty much #ifdef MS_WIN32 - #else.
Sorry for generalizing, "all" indeed is not the right word. My point is that we
should set expectation corr
Vlad Riscutia added the comment:
Changing title and type to better reflect issue.
On Windows MSVC build, ctypes is not correctly setting bitfields backed by 64
bit integers if specifying custom width. Simple repro:
from ctypes import *
class X(Structure):
_fields_ = [("a", c
Vlad Riscutia added the comment:
Terry, I am kind of new to this so I might not have marked header correctly.
Please feel free to fix it.
This is 100% bug not feature request. Initial message was confusing because
highstar thought Python has such bitfields as readonly and was asking for a
Vlad Riscutia added the comment:
Attached is addition to unittests which repro the issue. They will currently
pass on Linux but fail on Windows.
--
keywords: +patch
Added file: http://bugs.python.org/file22447/issue6068_unittest.diff
___
Python
Vlad Riscutia added the comment:
I took a look at this and I believe behavior is correct on Windows, the issue
is with the test. For example this test is failing:
class closest_fit(ctypes.BigEndianStructure):
_pack_ = 1# aligned to 8 bits, not ctypes default of 32
_fields_
Vlad Riscutia added the comment:
Attaching patch as previous attachment is only unittest. I included change to
SWAP_8 macro also as it looks like same issue and it will probably popup later.
--
Added file: http://bugs.python.org/file22451/issue6068_patch.diff
New submission from Vlad Riscutia :
Opened this issue to track configurable bitfield allocation strategy. This will
address issues like http://bugs.python.org/issue6069,
http://bugs.python.org/issue11920.
Summary: the way bitfields are allocated is up to the compiler not defined by
standard
Vlad Riscutia added the comment:
Opened http://bugs.python.org/issue12528 to address this.
--
nosy: +vladris
___
Python tracker
<http://bugs.python.org/issue11
Vlad Riscutia added the comment:
Opened http://bugs.python.org/issue12528 to address this.
--
versions: +Python 3.3
___
Python tracker
<http://bugs.python.org/issue6
Vlad Riscutia added the comment:
Removed previously attached partial patch, this is complete patch.
Summary:
Added following 3 constants in ctypes:
ctypes.BITFIELD_ALLOCATION_NATIVE
ctypes.BITFIELD_ALLOCATION_GCC
ctypes.BITFIELD_ALLOCATION_MSVC
Setting _bitfield_allocation_ attribute to one
Changes by Vlad Riscutia :
Removed file: http://bugs.python.org/file22617/cfield_bitfield_refactoring.diff
___
Python tracker
<http://bugs.python.org/issue12528>
___
___
Vlad Riscutia added the comment:
Looks like this was implemented by design at some point. In cfield.c, we have
specific code to treat character array fields as strings:
/* Field descriptors for 'c_char * n' are be scpecial cased to
return a Python string instead o
Vlad Riscutia added the comment:
I have a similar patch for issue 6068. I wasn't aware of this issue when I
looked into it.
I believe both patches fix the same thing (please take a look and correct me if
I'm wrong). My fix: we don't need to treat Windows differently, just rem
Vlad Riscutia added the comment:
I ran full test suit after making the _array_type = type(Array) change and
everything passes.
I also took a look at this and found additional leak. gc shows this as garbage:
[(,), , , , (, , , ), {'__dict__': , '_type_': '
Vlad Riscutia added the comment:
Added unit test to reproduce the issue (covers both big endian and little
endian structures so _other_endian function is hit on any machine).
Test currently fails without fix and passes with proposed fix in place.
--
keywords: +patch
nosy: +vladris
Vlad Riscutia added the comment:
Also added diff for fix:
- Implemented proposed issubclass(typ, Structure) solution
- Refactored _other_endian function because we used to getattr, catch
exception, then check if type is array/structure. I believe exception throwing
should not be on normal
Changes by Vlad Riscutia :
Removed file: http://bugs.python.org/file22627/issue4376_fix.diff
___
Python tracker
<http://bugs.python.org/issue4376>
___
___
Python-bug
Vlad Riscutia added the comment:
New patch containing unittest that actually tests the feature.
I would appreciate it if someone can try this on a bigendian machine.
--
Added file: http://bugs.python.org/file22642/issue4376_fix.diff
___
Python
Changes by Vlad Riscutia :
Removed file: http://bugs.python.org/file22642/issue4376_fix.diff
___
Python tracker
<http://bugs.python.org/issue4376>
___
___
Python-bug
Vlad Riscutia added the comment:
Changed c_int in tests to c_int32 just to be on the safe side.
--
Added file: http://bugs.python.org/file22643/issue4376_fix.diff
___
Python tracker
<http://bugs.python.org/issue4
Vlad Riscutia added the comment:
But if you set raw memory to let's say b'\0\0\0\1', when you look at the c_int
value afterwards, won't it be different on little endian and big endian
machines?
--
___
Python tracker
<
Changes by Vlad Riscutia :
Removed file: http://bugs.python.org/file22643/issue4376_fix.diff
___
Python tracker
<http://bugs.python.org/issue4376>
___
___
Python-bug
Vlad Riscutia added the comment:
You're right. I was busy swapping bytes in my head and missed that :)
--
Added file: http://bugs.python.org/file22644/issue4376_fix.diff
___
Python tracker
<http://bugs.python.org/i
Vlad Riscutia added the comment:
Ping? This also fixes 6493 (I believe in a cleaner way)
--
___
Python tracker
<http://bugs.python.org/issue6068>
___
___
Pytho
Vlad Riscutia added the comment:
Updated patch to reflect review feedback. Allocation strategy is now specified
as string in Python code.
I kept asserts in CanContinueField/CanExpandField because, as I said, default
case should never be hit. Input is validated in stgdict and this should make
Vlad Riscutia added the comment:
Reason you are seeing the failure for this is following change from 2.5 in
mmapmodule.c (:1109):
m_obj->data = (char *) MapViewOfFile(m_obj->map_handle,
dwDesiredAccess,
Vlad Riscutia added the comment:
How come tokenizer module is not based on actual C tokenizer? Wouldn't that
make more sense (and prevent this kind of issues)?
--
nosy: +vladris
___
Python tracker
<http://bugs.python.org/is
Vlad Riscutia added the comment:
The main reason for this issue is that internally ctypes doesn't consider
c_char_p as a pointer type. This is a bit counter-intuitive, but c_char_p is
treated as a simple type and has some other sugar built-in, like easier
integration with Python st
Vlad Riscutia added the comment:
Not sure how this can be fixed actually. I don't think PyErr_SetFromErrno
instead of PyErr_SetString would work because as far as I can tell, standard
does not mention dlopen making use of errno. Unlike Windows, where LoadLibrary
will change last-erro
Vlad Riscutia added the comment:
Attached patch for this issue.
This only happens on MSVC x64 (I actually tired to repro on Arch Linux x64
before starting work on it and it didn't repro).
What happens is that MSVC on x64 always passes structures larger than 8 bytes
by reference. See
Vlad Riscutia added the comment:
Changing type to behavior as it doesn't crash on 3.3. I believe issue was
opened against 2.6 and Santoso changed it to 2.7 and up where there is no crash.
Another data point: there is similar fix in current version of libffi here:
https://github.com/at
Vlad Riscutia added the comment:
Attached patch for 3.3 with unittest
--
keywords: +patch
nosy: +vladris
Added file: http://bugs.python.org/file22923/issue12764_patch3x.diff
___
Python tracker
<http://bugs.python.org/issue12
Vlad Riscutia added the comment:
Also patch for 2.7 with unittest. BTW, b"x" works on 2.7.
--
Added file: http://bugs.python.org/file22924/issue12764_patch2x.diff
___
Python tracker
<http://bugs.python.o
Vlad Riscutia added the comment:
Attached new mapping though I don't know where unit test for this should go...
--
keywords: +patch
nosy: +vladris
Added file: http://bugs.python.org/file22969/issue12802.diff
___
Python tracker
Vlad Riscutia added the comment:
Attached unit test.
--
Added file: http://bugs.python.org/file23026/issue12802_unittest.diff
___
Python tracker
<http://bugs.python.org/issue12
Vlad Riscutia added the comment:
I wasn't aware this is an auto-generated file. I can add a comment but looking
at it, it seems we auto-generate this file just to save a call to _dosmaperr. I
would refactor the whole function to call _dosmaperr first then if result is
still EINVAL,
Vlad Riscutia added the comment:
Oh, got it. Interesting. Then should I just add a comment somewhere or should
we resolve this as Won't Fix?
--
___
Python tracker
<http://bugs.python.org/is
Vlad Riscutia added the comment:
Attached updated patch which extends generrmap.c to allow for easy addition of
other error mappings.
Also regenerated errmap.h and unittest.
--
Added file: http://bugs.python.org/file23054/issue12802_2.diff
Vlad Riscutia added the comment:
Ah, I see Antoine already attached a patch. I was 3 minutes late :)
--
___
Python tracker
<http://bugs.python.org/issue12
Vlad Riscutia added the comment:
Meador, I believe this was the first issue on the tracker that got me looking
into bitfield allocation. I agree that big-endian on MSVC doesn't make too much
sense but you can disregard that - using default endianess will still yield
different siz
Vlad Riscutia added the comment:
Sounds good. Please nosy me in the doc bug.
--
___
Python tracker
<http://bugs.python.org/issue6069>
___
___
Python-bugs-list m
Vlad Riscutia added the comment:
Well currently we pack bitfields with an algorithm that uses #ifdefs for GCC
and MSVC builds. This feature tries to remove the hardcoded behavior and
implement it as a runtime option. This should improve interop with other
compilers. Currently I provided
Vlad Riscutia added the comment:
I believe there is a deeper issue here in ctypes design. Basically we provide
both c_char_p and POINTER(c_char) which should behave exactly the same since
both are the equivalent of char* in C but internally they have different
implementations.
c_char_p is
Vlad Shcherbina added the comment:
1. "This patch adds too many lines of code and not enough value."
If the maintainers judge it so, I have nothing to say. You have the
responsibility to keep the codebase relatively simple.
2a. "Existing programs made with the assumption th
New submission from Vlad Emelianov :
Lib2to3 does not support changes made in https://bugs.python.org/issue32117
```python
def test():
my_list = ["value2", "value3"]
yield "value1", *my_list
return "value1", *my_list
```
The idea is to use `
Change by Vlad Emelianov :
--
versions: +Python 3.8
___
Python tracker
<https://bugs.python.org/issue38641>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Vlad Hoi :
from abc import ABC
class A:
def __init_subclass__(self):
pass
class B(ABC, A, name="name"):
pass
After initialising class B, this exception occurs, because multiple "name"
arguments where provided:
Traceback (most recen
Change by Vlad Hoi :
--
keywords: +patch
pull_requests: +24117
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/25385
___
Python tracker
<https://bugs.python.org/issu
New submission from Vlad Emelianov :
Add changes made in https://github.com/python/typing/pull/700 to upstream.
--
components: Library (Lib)
messages: 361957
nosy: Vlad Emelianov
priority: normal
severity: normal
status: open
title: Fix TypedDict totalizy check for inherited keys
type
Change by Vlad Emelianov :
--
keywords: +patch
pull_requests: +17879
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/18503
___
Python tracker
<https://bugs.python.org/issu
Change by Vlad Emelianov :
--
title: Fix TypedDict totalizy check for inherited keys -> Fix TypedDict
totality check for inherited keys
___
Python tracker
<https://bugs.python.org/issu
New submission from Vlad Shcherbina :
To reproduce:
1. Download and run Python installer (I used python-3.7.3-amd64-webinstall.exe).
2. Modify install settings:
- Install for all users: yes
- Customize install location: "C:\Python37"
(don't know if it's relevant, m
Vlad Shcherbina added the comment:
https://github.com/pypa/pip/issues/6582
--
___
Python tracker
<https://bugs.python.org/issue37204>
___
___
Python-bugs-list m
New submission from Vlad Shcherbina:
Toy example:
>>> from collections import namedtuple
>>> Rectangle = namedtuple('Rectangle', 'width height')
>>> Ellipse = namedtuple('Ellipse', 'x_axis y_axis')
>>> Rectangle(width
Vlad Shcherbina added the comment:
While we are at it, namedtuple inherits other operations that make no sense for
fixed-length tuples:
>>> Rectangle(width=1, height=2) * 3
(1, 2, 1, 2, 1, 2)
>>> Rectangle(width=1, height=2) + Ellipse(x_axis=3, y_axis=4)
(1, 2, 3, 4)
Bu
Vlad Tudorache added the comment:
I confirm the issue. Screenshot attached.
--
type: -> behavior
Added file: https://bugs.python.org/file47987/Capture d’écran 2018-12-11 à
22.07.30.png
___
Python tracker
<https://bugs.python.org/issu
Vlad Tudorache added the comment:
At the line 102 in editor.py, I see:
self.top = top = window.ListedToplevel(root, menu=self.menubar)
Looking at window.py it seems that ListedToplevel builds a new Toplevel (the
black one?) instead of adding an existing one (the editor) at the list, I&#x
Vlad Tudorache added the comment:
No, I'm wrong, the editor window seems created by the ListedToplevel.
--
___
Python tracker
<https://bugs.python.org/is
Vlad Tudorache added the comment:
The only versions of Tk not showing issues on my Mac are:
- 8.5.18 with any Python in 3.5, 3.6, 3.7 on Mojave AND previous versions;
- 8.6.8 when built on a 10.13 SDK, with the same Python versions.
For 8.6.8 built on Mojave SDK, there's the black backg
New submission from Vlad Shcherbina :
To reproduce, run the following program:
import sqlite3
conn = sqlite3.connect(':memory:')
conn.row_factory = sqlite3.Row
print(conn.execute("SELECT 'John' AS name, 42 AS salary").fetchone())
It prints ''.
It wo
Vlad Shcherbina added the comment:
There is no need to add explicit knowledge of reprlib to the Row object or vice
versa.
Those who use reprlib to limit output size will have no problem. Reprlib
already truncates arbitrary reprs at the string level:
https://github.com/python/cpython/blob
Vlad Shcherbina added the comment:
Any chance this could get into 3.7.3?
--
nosy: +vlad
___
Python tracker
<https://bugs.python.org/issue34776>
___
___
Pytho
New submission from Vlad Shcherbina :
from typing import *
def f(arg: str = None):
pass
print(get_type_hints(f))
# {'arg': typing.Union[str, NoneType]}
# as expected
class T(NamedTuple):
field: str = None
print(get_type_hints(T))
# {'field': }
# but it
Vlad Shcherbina added the comment:
If the maintainers agree that it's desirable to automatically deduce
optionality, I'll implement it.
--
___
Python tracker
<https://bugs.python.o
New submission from Vlad Starostin :
Long story short, if you have a TCP asyncio-server (implementing
asyncio.Protocol) that sends something to socket in connection_made callback,
it will leak fds and won't be able to handle some consequent requests, if you
will send to it RST right afte
Change by Vlad Starostin :
--
keywords: +patch
pull_requests: +6146
stage: -> patch review
___
Python tracker
<https://bugs.python.org/issue33263>
___
___
Py
Vlad Tudorache added the comment:
I can reproduce this problem only with Tk 8.6.7, both compiled by myself or
installed from ActiveState (Python 3.7 and 3.6 compiled by myself, too). I
can't see it with Tk 8.6.8 provided with the installers, nor with Tk 8.5.18.
--
nosy: +vtudo
New submission from Vlad Tudorache :
When closing the IDLE's Preferences dialog the IDLE's Console window and/or
Editor window (whichever was active before) doesn't show cursor/caret any more.
One must click on Desktop or another window then click again in the IDLE's
Vlad Tudorache added the comment:
I’m sorry for not being clear. This problem appears with Tk 8.6, even in
Python 3.5 (compiled by me).
I’ve succeeded to reproduce only once the @autorelease pool error (it
appeared after several open/close of the preferences or about window). Did
anyone else
Vlad Tudorache added the comment:
I can confirm that removing the grab_set() calls fixes the locking. But am I
the only one to notice that the dialogs aren't modal any more?
--
___
Python tracker
<https://bugs.python.org/is
Vlad Tudorache added the comment:
Adding self.grab_release() in the ok() method for the dialogs BEFORE destroy()
solved the problem for me, keeping the grab_set() methods in place
(uncommented, actives).
--
___
Python tracker
<ht
Vlad Tudorache added the comment:
I've checked again the source code in config_key.py, configdialog.py,
help_about.py, query.py, searchbase.py, textview.py and added a
self.grab_release() in functions like ok() and cancel() before the call to
self.destroy() and I see no more lock/
Vlad Tudorache added the comment:
I confirm the crashes. I've tried with personal builds of python 3.5, 3.6, 3.7,
with ActiveTcl and personal builds of Tcl 8.6.x, the problem does not appear
with 8.5.18, but shows itself with 8.5.19 (strange, API changes?). I'll try to
i
Vlad Tudorache added the comment:
The problem shows itself on macOS High Sierra in some Tk Demos, independently
of Python's IDLE. So there is something completely different, to report to the
Tcl/Tk community. Strangely, I didn't notice the issues on Sierra, but I have
no Sierra
Vlad Tudorache added the comment:
I was telling only that I've seen myself the issues with some Tcl/Tk versions.
As I often build Tcl/Tk on macOS, I can confirm that on 8.6.8 I didn't see
issues (those were effectively present in (ActiveState or not) Tcl 8.5.18,
8.5.19, 8.6.6,
Vlad Tudorache added the comment:
Having 3.6.5 and 3.7.0 with Tcl/Tk 8.6.8, try to File->Open Module and open ssl
module. On 3.6 everything is fine, on 3.7 the scroller sticks at the bottom. A
Tk text widget with the same amount of lines shows no problem. Apple macOS
17.7.0 with 64
Vlad Tudorache added the comment:
The solution was very simple on my Mac. In
/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/idlelib/editor.py,
lines 461-462 should be:
up = {EventType.MouseWheel: (event.delta >= 0) == darwin,
EventType.Button: event.num == 4}
inst
Vlad Tudorache added the comment:
With:
up = {EventType.MouseWheel: (event.delta >= 0) == darwin,
EventType.Button: (event.num == 4)}
in editor.py at 461-462 I don't see bugs #1 neither #3 on 3.7.0 but I'll try
wit
Vlad Tudorache added the comment:
In fact, if the first click before dragging takes place in the upper half of
the scrollbar slider, the click is not received nor the drag after. Is like the
scroll slider is translated down with half of the slider length (that's why I
can drag the s
Vlad Tudorache added the comment:
Check the commands below and the results to see why the problem with the mouse
wheel appeared:
>>> a = 1
>>> a >= 0 == True
False
>>> (a >= 0) == True
True
>>> a >= (0 == True)
True
>>> a >= 0 == Fa
New submission from Vlad Tudorache :
I've just started using the embedded Python 3.6 for Windows and I find
turtle.py in the archive, knowing that tkinter isn't present. There's no large
space loss because of it, but it's presence may be confusing.
--
comp
New submission from Vlad Starostin :
The docstring says "This may be None or an instance of EventLoop". But docs
explicitly state that get_event_loop "must never return None". The same
docstring is also in the example in docs:
https://docs.python.org/3.6/library/asy
Vlad Tudorache added the comment:
Thank you very much. Then I will simply remove it.
--
___
Python tracker
<https://bugs.python.org/issue34343>
___
___
Pytho
Vlad Tudorache added the comment:
I've tried to check the source code of IDLE in search of chained comparisons
without parenthesis (like the example I showed and the bug with the mouse
wheel). I couldn't find something important.
Then I wrote in a file the following code:
impo
Vlad Tudorache added the comment:
Edit:
The code is:
import tkinter
root = tkinter.Tk()
text = tkinter.Text(root)
vbar = tkinter.Scrollbar(root)
vbar.pack(side=tkinter.RIGHT, fill=tkinter.Y)
text.pack(side=tkinter.LEFT, fill=tkinter.BOTH, expand=1)
text.config(yscrollcommand=vbar.set
Vlad Tudorache added the comment:
And the result (video) of my script is attached to this post.
--
Added file: https://bugs.python.org/file47738/tkinter_scroll_issues.mov
___
Python tracker
<https://bugs.python.org/issue34
Vlad Tudorache added the comment:
The scroll works. Many thanks, I thought the callback should have been
rewritten, too. Should one open a different report for the clicks on the
scrollbar ends, like in the script I put (independent of IDLE
New submission from Vlad Tudorache :
Run the python script below.
import tkinter
root = tkinter.Tk()
text = tkinter.Text(root)
vbar = tkinter.Scrollbar(root)
vbar.pack(side=tkinter.RIGHT, fill=tkinter.Y)
text.pack(side=tkinter.LEFT, fill=tkinter.BOTH, expand=1)
text.config(yscrollcommand
Change by Vlad Tudorache :
--
stage: -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.org/issue34343>
___
___
Python-bugs-list
Vlad Tudorache added the comment:
The bug needs forwarding to the Tcl/Tk community. This is the script written in
Tcl (run with tclsh8.6 script.tcl for Tcl 8.6 and tclsh8.5 script.tcl for 8.5):
package require Tk
scrollbar .vbar -width 10
text .edit
pack .vbar -side right -fill y
pack
Vlad Tudorache added the comment:
Now, the video for TK 8.5 showing expected behavior on macOS.
--
Added file: https://bugs.python.org/file47743/Tk85Mac.mov
___
Python tracker
<https://bugs.python.org/issue34
Vlad Tudorache added the comment:
It seems that Kevin's fix solves the issues.
--
___
Python tracker
<https://bugs.python.org/issue34370>
___
___
Pytho
1 - 100 of 140 matches
Mail list logo
