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 brian.curtin, enolte, loewis, stutzbach, techtonik
Date 2010-04-28.04:17:00
SpamBayes Score 0.095577866
Marked as misclassified No
Message-id <>
Thank for the feedback.  I'll revise the patch tomorrow based on your code-cleanup suggestions.

To answer your question about the thread and explain why the test sometimes passes:

The goal of test_changing_value is to try to trigger a race condition that exists inside many of these function.  So, yes, the thread is necessary. :) Thankfully the functions call Py_BEGIN_ALLOW_THREADS; otherwise, I'd have to launch another process to try to trigger it.

Sometimes we never manage to hit the race condition and the test passes even though the bug is present.  I can increase the number of attempts to trigger it (currently "for _ in range(100)") to make it more likely to occur every time the test is run.
Date User Action Args
2010-04-28 04:17:03stutzbachsetrecipients: + stutzbach, loewis, techtonik, enolte, brian.curtin
2010-04-28 04:17:02stutzbachsetmessageid: <>
2010-04-28 04:17:01stutzbachlinkissue2810 messages
2010-04-28 04:17:00stutzbachcreate