Message 106636 - Python tracker

➜

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	belopolsky
Recipients	amaury.forgeotdarc, belopolsky, loewis, mark.dickinson, pitrou, typo.pl, vstinner
Date	2010-05-28.04:46:41
SpamBayes Score	0.000644801
Marked as misclassified	No
Message-id	<1275022007.27.0.197724679583.issue4653@psf.upfronthosting.co.za>
In-reply-to

Content
I don't run Windows either but I believe sizeof(char) == 1 is guaranteed by the C standard, so the first report (Python/dynload_win.c) is invalid. The last two appear to be valid [1], and a common mistake. [2] Unit tests are needed for these. I also notice that r81156 does not add a unit test either. ISTM, a test case can be crafted by setting sys.stderr to None. [1] http://msdn.microsoft.com/en-us/library/aa366537(VS.85).aspx [2] http://blogs.msdn.com/b/oldnewthing/archive/2004/03/02/82639.aspx

I don't run Windows either but I believe sizeof(char) == 1 is guaranteed by the C standard, so the first report (Python/dynload_win.c) is invalid.

The last two appear to be valid [1], and a common mistake. [2]  Unit tests are needed for these.  I also notice that r81156 does not add a unit test either.  ISTM, a test case can be crafted by setting sys.stderr to None.

[1] http://msdn.microsoft.com/en-us/library/aa366537(VS.85).aspx
[2] http://blogs.msdn.com/b/oldnewthing/archive/2004/03/02/82639.aspx

History
Date	User	Action	Args
2010-05-28 04:46:47	belopolsky	set	recipients: + belopolsky, loewis, amaury.forgeotdarc, mark.dickinson, pitrou, vstinner, typo.pl
2010-05-28 04:46:47	belopolsky	set	messageid: <1275022007.27.0.197724679583.issue4653@psf.upfronthosting.co.za>
2010-05-28 04:46:44	belopolsky	link	issue4653 messages
2010-05-28 04:46:42	belopolsky	create