Message190634
| Author |
terry.reedy |
| Recipients |
gpolo, terry.reedy |
| Date |
2013年06月05日.00:10:09 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1370391011.13.0.497948968214.issue4343@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I am still confused about the master/parent business and why required args are documented as optional. (Besides #18131, I also discovered that messageboxes usually, but not always, require parent=something passed, to be collected into **options. I need to read more before I can really review and think about applying this.
Why isn't _default_root initialized to something useful instead of None. If it were, _support_default_root would not be needed.
The new function is slightly different from the code it replaces. The main difference I see with Misc.setup is that your code does *not* stash Tk() in _default_root. Since every Tk() call creates a new screen window, I do not see how this is correct.
On the plus side, I believe your patch would fix part of the problem I reported in #18131. |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2013年06月05日 00:10:11 | terry.reedy | set | recipients:
+ terry.reedy, gpolo |
| 2013年06月05日 00:10:11 | terry.reedy | set | messageid: <1370391011.13.0.497948968214.issue4343@psf.upfronthosting.co.za> |
| 2013年06月05日 00:10:11 | terry.reedy | link | issue4343 messages |
| 2013年06月05日 00:10:09 | terry.reedy | create |
|