Author markroseman
Recipients Todd.Rovito, ezio.melotti, markroseman, mcgrete, ned.deily, roger.serwy, serhiy.storchaka, taleinat, terry.reedy
Date 2015-07-30.14:52:06
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1438267927.06.0.345571049903.issue13884@psf.upfronthosting.co.za>
In-reply-to
Content
Doing it via the option database vs. on each menu would be my preferred approach. The option database is global, not per toplevel, so would cover everything. Only 'downside' is the whole its-not-an-application-its-a-library thing, though in the highly unlikely case someone was actually using idlelib this way, I'd consider setting that option a public service.

There's no harm doing it the other way (and I was being polite since the previous patch was done that way), but it does very marginally increase maintainability difficulties (remembering to keep the option and add it to any future menus).

Either way - please let's just make this change! :-)
History
Date User Action Args
2015-07-30 14:52:07markrosemansetrecipients: + markroseman, terry.reedy, taleinat, ned.deily, ezio.melotti, roger.serwy, Todd.Rovito, mcgrete, serhiy.storchaka
2015-07-30 14:52:07markrosemansetmessageid: <1438267927.06.0.345571049903.issue13884@psf.upfronthosting.co.za>
2015-07-30 14:52:07markrosemanlinkissue13884 messages
2015-07-30 14:52:06markrosemancreate