Message73749
I'm opposed to this patch. As a design principle, any class should
always call its base class' __init__, unless that is known to be empty.
With this patch, the base __init__ call becomes conditional, which hints
at a design bug of the entire class hierarchy.
I don't quite understand *what* the problem is. If Screen is intended to
be a singleton, then the right fix would be to make Screen a function,
which maintains a singleton reference, and Screen be renamed to _Screen.
The fact that the same design bug already exists in 3.0 doesn't mean it
should be backported to 2.6. |
|
Date |
User |
Action |
Args |
2008-09-24 18:59:15 | loewis | set | recipients:
+ loewis, georg.brandl, gregorlingl, benjamin.peterson |
2008-09-24 18:59:15 | loewis | set | messageid: <1222282755.52.0.237846008565.issue3956@psf.upfronthosting.co.za> |
2008-09-24 18:59:11 | loewis | link | issue3956 messages |
2008-09-24 18:59:11 | loewis | create | |
|