[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Alexander Belopolsky added the comment: See also "turtle.py update for 3.1" post on python-dev specifically mentioning this bug as fixed. http://mail.python.org/pipermail/python-dev/2009-May/089383.html -- status: pending -> closed ___ Python track

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Alexander Belopolsky added the comment: It looks like this has been fixed in turtle 1.1. See r72318 and issue 5923. -- nosy: +belopolsky, georg.brandl resolution: -> out of date stage: -> committed/rejected status: open -> pending ___ Python track

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: Sorry, but there is a remain from testing different versions of the turtle module in the demo file's import statement. Should read (of course): from turtle import Screen, Turtle, mainloop A corrected version is attached Sorry, again, for the

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: Here the patch, updated Added file: http://bugs.python.org/file12080/setup_patch.diff ___ Python tracker <[EMAIL PROTECTED]> ___ ___

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: The attached file setup_bug_demo.py shows the bug of this issue. It's taken out of a working game application and radically abridged to concentrate on this issue. Now the patch to fix the bug has to be changed a bit (because of the changes of M

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: > Most definitely. The module went into Python without any review > whatsoever. Nobody (but you) has ever looked at the code in detail. That's not True! Brad Miller, for example, who also had submitted patches to the pythontracker, coauthor of

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Martin v. Löwis <[EMAIL PROTECTED]> added the comment: > 2. Is it really fruitful to discuss general design issues along with > (comparatively) small problems like this one - in the sense of > alternative ways to fix that problem? Most definitely. The module went into Python without any review

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: > > I find that desire misguided; this is (IMO) a case of false abstraction. > > Is there any kind of proof that this design actually > > works, i.e. can be ported to a different GUI library (like, say, > > PythonWin? or AWT, when run in Jython?

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Martin v. Löwis <[EMAIL PROTECTED]> added the comment: > I cannot call the Canvas method _update_idletasks() from within > _Screen.setup() becaus this would contradict to the architecture of the > module which isolates all direct references to Tkinter to > TurtleScreenBase. (The idea behind this

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: I cannot call the Canvas method _update_idletasks() from within _Screen.setup() becaus this would contradict to the architecture of the module which isolates all direct references to Tkinter to TurtleScreenBase. (The idea behind this is to make

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Guilherme Polo <[EMAIL PROTECTED]> added the comment: Why are you using update instead of update_idletasks ? I'm not talking exactly about this line being added, but this ends up calling TurtleScreenBase._update which calls self.cv.update(), cv being a canvas. update_idletasks should be used exa

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: I'd also like to see this patch accepted and done for Python 2.6.1 and (at the same time) python 3.0 before the last rc is released. So perhaps you could you dedicate a few minutes to this one also. Thanks for your efforts, Gregor

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

Gregor Lingl <[EMAIL PROTECTED]> added the comment: This patch applies to Python 2.6 as well Gregor -- versions: +Python 2.6 ___ Python tracker <[EMAIL PROTECTED]> ___

[issue4117] missing update() in _Screen.setup() of Lib/turtle.py

New submission from Gregor Lingl <[EMAIL PROTECTED]>: In Lib/turtle.py _Screen.setup() needs a final update() call in order to ensure, that immediately following calls of window_width() or window_height() return correct values. Consequently the setup() method call in _Screen.__init__() has to t