This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author gpolo
Recipients gpolo
Date 2009-06-06.21:56:31
SpamBayes Score 1.4820519e-07
Marked as misclassified No
Message-id <1244325396.24.0.788727463424.issue6225@psf.upfronthosting.co.za>
In-reply-to
Content
Hi,

While testing Tkinter.Canvas I've found several minor bugs that I would
prefer to see fixed. Many of them change the current Canvas api a bit,
but for better. For example, the methods "focus", "gettags", "icursor",
"index", "insert", "move" (and some others) accept arbitrary amount of
arguments, but all these tcl subcommands have a fixed amount of
arguments they accept, so I consider it is better to make this clear on
Tkinter too.

I've also found a problem in Misc._configure which is also fixed by the
attached patch. The problem is that when cnf is a string, the call
"self.tk.split(self.tk.call(_flatten((self._w, cmd, '-'+cnf))))" may
still result in an empty string causing the following statement to fail
"return (x[0][1:],) + x[1:]".

One thing that left me curious was the comment "# XXX Should use
_flatten on args" in Canvas.coords. I've tried understanding why it
should use _flatten there, but couldn't figure it out. This is a very
old comment, so maybe it is no longer true ?
History
Date User Action Args
2009-06-06 21:56:36gpolosetrecipients: + gpolo
2009-06-06 21:56:36gpolosetmessageid: <1244325396.24.0.788727463424.issue6225@psf.upfronthosting.co.za>
2009-06-06 21:56:34gpololinkissue6225 messages
2009-06-06 21:56:33gpolocreate