Daniel Stutzbach <dan...@stutzbachenterprises.com> added the comment:
I'm only setup to test the official Windows build setup under PCbuild. I'm not testing the legacy build setups under PC/VC6, PC/VS7.1, or PC/VS8.0. The patch against the trunk failed for PC/VC6/pythoncore.dsp. I don't need that to build, though. Your patch built fine, with one warning about a loss of precision on the following line in sinpi(): n = round(2.0*y); I recommend explicitly casting the result of round() to int, to get rid of the warning and because explicit is better than implicit. :) I ran the following tests, all of which passed: PCbuild/python.exe Lib/test/regrtest.py -v test_math I appreciate your work on this patch. Let me know if there's anything else I can do. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue3366> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com