Author giampaolo.rodola
Recipients giampaolo.rodola, neologix, vstinner, yselivanov
Date 2017-04-07.18:09:29
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <>
In certain protocols modify() is supposed to be used on every interaction between client and server. E.g. an FTP server does this:

- register(fd, EVENT_READ); recv()         # wait command from client
- modify(fd, EVENT_WRITE); send(response)  # send response
- modify(fd, EVENT_READ); recv()           # wait for new command it's two calls for each command received.
In asyncio modify() is also used in different circumstances. 
If you're worried about code duplication I can refactor first, but IMO it would be a bad idea to reject this or future improvements because the current code status prevents code reuse. Right now there are already 3 classes sharing basically the same code (poll, epoll and devpoll related classes). Those can be refactored similarly to this:
Date User Action Args
2017-04-07 18:09:29giampaolo.rodolasetrecipients: + giampaolo.rodola, vstinner, neologix, yselivanov
2017-04-07 18:09:29giampaolo.rodolasetmessageid: <>
2017-04-07 18:09:29giampaolo.rodolalinkissue30014 messages
2017-04-07 18:09:29giampaolo.rodolacreate