Mark Dickinson <dicki...@gmail.com> added the comment:

Patch looks fine to me.  A couple of minor points (which I'm happy to fix at 
commit time if you agree, and don't want to create a new patch):

- it looks to me as though the #ifdef _WIN32 isn't necessary any more;  it's 
enough that we're on MSVC and that this isn't a 64-bit build.

- I'd suggest replacing _PC_53 with _PC_53 | _RC_NEAR.  I know it's technically 
redundant, since _RC_NEAR happens to be 0, but it makes the intent of the code 
clearer.

I'd still want to see all the tests pass on both x64 and x86 Windows before 
applying this;  I may have an opportunity to test this on x64 in the near 
future.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue13889>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to