This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients Nir Soffer, giampaolo.rodola, serhiy.storchaka, vstinner, walkhour
Date 2017-07-25.22:14:30
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1501020870.84.0.0209426689677.issue30985@psf.upfronthosting.co.za>
In-reply-to
Content
"Giampaolo, people using only 3.7 should probably use asyncio. Fixing
asyncore is more important to those that can use only 2.7 (e.g.Centos/RHEL) or have to support both python 3 and 2."

IMHO starting to use closing in asyncore *is* a backward incompatible change. We don't know how applications use this existing attribute. Maybe it's not set to a boolean. Maybe it uses a different policy.

I would be ok to start using closing in Python 3.7, but I'm not confortable with backporting such change.

I would be if would be add a new private attribute, as the proposed "_closed" name.

Maybe nobody uses closing. Maybe people using closing have a similar usage. The thing is that we don't know, and according to what you wrote Nir, closing *is* used.

What I dislike in asyncore is that subclassing is not prohibed, it seems wanted by design. Ok, but what if a subclass overrides completely a method and doesn't set "closing/_closed" anymore? That's another reason why I dislike the idea of making any change in asyncore *especially* in Python 2.7 which is now considered as "very stable".
History
Date User Action Args
2017-07-25 22:14:30vstinnersetrecipients: + vstinner, giampaolo.rodola, serhiy.storchaka, Nir Soffer, walkhour
2017-07-25 22:14:30vstinnersetmessageid: <1501020870.84.0.0209426689677.issue30985@psf.upfronthosting.co.za>
2017-07-25 22:14:30vstinnerlinkissue30985 messages
2017-07-25 22:14:30vstinnercreate