Author ncoghlan
Recipients Robin.Schreiber, amaury.forgeotdarc, asvetlov, benjamin.peterson, brett.cannon, eric.snow, ezio.melotti, loewis, ncoghlan, ned.deily, pitrou, serhiy.storchaka, yselivanov
Date 2015-02-28.11:44:43
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1425123883.9.0.747160850662.issue20204@psf.upfronthosting.co.za>
In-reply-to
Content
The case for RuntimeWarning: the object isn't necessarily *broken* as such (most things will still work), but pickling and some introspection features may not work properly.

The case for DeprecationWarning: breaking picking and introspection is bad when the fix (setting __module__) is straightforward, so this should eventually become an AttributeError:

    AttributeError: __module__ not set on builtin type tkapp

My own preference is for the latter - eventually making it a hard requirement to specify the module name properly. Even true builtins officially live in the builtins module:

    >>> str.__module__
    'builtins'
History
Date User Action Args
2015-02-28 11:44:43ncoghlansetrecipients: + ncoghlan, loewis, brett.cannon, amaury.forgeotdarc, pitrou, benjamin.peterson, ned.deily, ezio.melotti, asvetlov, eric.snow, serhiy.storchaka, Robin.Schreiber, yselivanov
2015-02-28 11:44:43ncoghlansetmessageid: <1425123883.9.0.747160850662.issue20204@psf.upfronthosting.co.za>
2015-02-28 11:44:43ncoghlanlinkissue20204 messages
2015-02-28 11:44:43ncoghlancreate