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 stutzbach
Recipients mark.dickinson, nirinA, rhettinger, stutzbach, terry.reedy
Date 2009-09-21.13:47:33
SpamBayes Score 1.6412159e-08
Marked as misclassified No
Message-id <1253540856.03.0.57638778958.issue3366@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2009-09-21 13:47:36stutzbachsetrecipients: + stutzbach, rhettinger, terry.reedy, mark.dickinson, nirinA
2009-09-21 13:47:36stutzbachsetmessageid: <1253540856.03.0.57638778958.issue3366@psf.upfronthosting.co.za>
2009-09-21 13:47:33stutzbachlinkissue3366 messages
2009-09-21 13:47:33stutzbachcreate