Alexander Belopolsky <[email protected]> added the comment:
Mark,
I have zero experience with Windows and don't even have a win32 machine to test
the patch.
On the other hand the patch is so simple that I think it can be reviewed based
on theoretical considerations.
This is probably bikesheding, but I have a slight preference for os.name in
("nt", "ce"). The reason is that sys.platform is fixed when python is built
while os.name is (in theory) determined at run-time.
Also, sys.platform == "win32", appears to be false on 64 bit Windows, but I
think it is actually true.
Finally, explicit better than implicit. A change from if os.name == "nt" to
os.name in ("nt", "ce") gives obviously a strictly wider check. On the other
hand it is not obvious to me how the current patch will affect Cygwin platform.
----------
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue7879>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
