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 pklanke
Recipients neologix, pitrou, pklanke, vstinner, yselivanov
Date 2017-09-26.11:42:14
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <afcecee5-f1dc-086d-1582-b694ec5c79de@protonic.nl>
In-reply-to <1506421795.52.0.933594631093.issue30844@psf.upfronthosting.co.za>
Content
On 26-09-17 12:29, STINNER Victor wrote:
> 
> STINNER Victor added the comment:
> 
> Using Git history, I found the following commit which added "r, w, x = select(r, w, w, timeout)" for Windows in tulip (old name of the asyncio project, when it was developed outside CPython):
> 
> commit 84124f3d725c9931249d083e78f43fcda91c383a
> Author: Richard Oudkerk <shibturn@gmail.com>
> Date:   Fri Jan 18 13:03:09 2013 +0000
> 
>      Minimal chages to make tests pass on Windows
> 
> https://github.com/python/asyncio/commit/84124f3d725c9931249d083e78f43fcda91c383a
Thanks!

Although this still does not teach us why the test failed on Windows and 
what is solved by adding the wrapper method, it does suggest that indeed 
the runtime behaviour will not be affected by removing the wrapper method.

The unit test might. To be honest, I did not run unit tests on Windows. 
Maybe by doing this, it might just tell us exactly what is accomplished 
with the wrapper method.
> 
> ----------
> 
> _______________________________________
> Python tracker <report@bugs.python.org>
> <https://bugs.python.org/issue30844>
> _______________________________________
>
History
Date User Action Args
2017-09-26 11:42:14pklankesetrecipients: + pklanke, pitrou, vstinner, neologix, yselivanov
2017-09-26 11:42:14pklankelinkissue30844 messages
2017-09-26 11:42:14pklankecreate