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 r.david.murray
Recipients cjwelborn, r.david.murray, tim.peters
Date 2014-01-31.01:18:02
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1391131084.02.0.98739587742.issue18916@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks for working on this.  The patch looks pretty good.  I would prefer to flow the two lines in the Lock docstring as a paragraph, rather than have once sentence per line.  Also, the OP mentions that the docs for type(Lock) mention PyThread_allocate_lock...do you want to look at fixing that as well?  As mentioned, it should reference Lock() instead.  The reference to PyThread_allocate_lock should be moved into comments in the type, IMO.

Also, this patch may be affected by the Argument Clinic Derby.  I don't know if the threading module is one of the ones that is being converted or not.  You can just ignore that issue since the full patch is needed for 3.3, but if you feel like figuring it out, and it is affected, you could produce an additional patch for 3.4.  Otherwise whoever applies your patch will figure out how to merge it.
History
Date User Action Args
2014-01-31 01:18:04r.david.murraysetrecipients: + r.david.murray, tim.peters, cjwelborn
2014-01-31 01:18:04r.david.murraysetmessageid: <1391131084.02.0.98739587742.issue18916@psf.upfronthosting.co.za>
2014-01-31 01:18:03r.david.murraylinkissue18916 messages
2014-01-31 01:18:02r.david.murraycreate