[issue3940] turtle.py - minor bugfixes

[Martin v. Löwis]

Martin v. Löwis <[EMAIL PROTECTED]> added the comment:

Please submit separate patches for unrelated issues. As a whole, the
patch is unacceptable because of that.

Please don't add comments to the code that describe what has changed, or
what was before. Only describe what is. Specifically, don't say
"renamed:...".

Also, for each issue, it would be good if you could provide a short test
case that demonstrates the bug that gets fixed with the patch.

--
nosy: +loewis
resolution:  -> rejected
status: open -> closed

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3940] turtle.py - minor bugfixes

[Gregor Lingl]

Gregor Lingl <[EMAIL PROTECTED]> added the comment:

The bugfix for bug 3) described above makes necessary the insertion of a
line in turtleDemo.py (around line 96)

Again I've attached the corresponding diff file

Added file: http://bugs.python.org/file11566/turtleDemo.diff

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3940] turtle.py - minor bugfixes

[Gregor Lingl]

New submission from Gregor Lingl <[EMAIL PROTECTED]>:

Thorough testing revealesd the following bugs in turtle.py (Python 2.6):

1) Around lines 359 and 379: There's a name conflict with a methodname
of the parentclass Frame: _root. The bugfix consists in renaming this
attribute, which occurs only twice

2) Around line 732: Turtles with image-shapes do not recognize user
coordinates. This is fixed in  TurtleScreenBase method _drawimage by
introducing the corresponding scaling factors

3) Around line 3570: Calling the __init__ - Method of the Screen-class
(which uses the Borg idiom) destroys graphics and Turtles on the Screen.
This is an undesired behaviour. The fix brings the behaviour of Screen
in accordance with the Python 3.0 version.

4) Around line 3612: Screen.setup() needs a final update() call in order
to ensure, that immediately following calls of window_width() or
window_height() return correct values.

Moreover there is a correction of the version number and date

Bugfixes are attached in turtle.py.diff

Regards,
Gregor

--
components: Library (Lib)
files: turtle.py.diff
keywords: patch
messages: 73607
nosy: gregorlingl
severity: normal
status: open
title: turtle.py - minor bugfixes
type: behavior
versions: Python 2.6
Added file: http://bugs.python.org/file11565/turtle.py.diff

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com