Martin v. Löwis <[email protected]> added the comment:

These look fine. I think further changes are necessary:
tkinter/__init__.py tries to load createfilehandler/deletefilehandler;
this becomes redundant. Also, why did you leave DoOneEvent and Quit
as-is - don't they fall into the same category?

One minor nit: I personally dislike function pointer casts, because they
can easily go wrong. I rather prefer if the C compiler tells me when I
mess up with function pointer types. Hence _tkinter uses selfptr/self
all the time.

----------
assignee:  -> gpolo

_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue3638>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to