Re: [Python-Dev] cpython: Fix email post-commit review comments.

2012-04-18 Thread Nick Coghlan
On Thu, Apr 19, 2012 at 12:21 AM, Antoine Pitrou wrote: > I don't think INCREFs were necessary, actually. > PyDict_SetItemString doesn't steal a reference. Yes, I was tired when that checkin went by and my brain didn't register that the function was otherwise using borrowed refs for name and path

Re: [Python-Dev] cpython: Fix email post-commit review comments.

2012-04-18 Thread Greg Ewing
On 19/04/12 11:22, Tres Seaver wrote: Maybe we should mandate that their names end with '_rtfm'. +1 -- Greg ___ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mai

Re: [Python-Dev] cpython: Fix email post-commit review comments.

2012-04-18 Thread Tres Seaver
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/18/2012 06:48 PM, Greg Ewing wrote: > Functions with ref-stealing APIs really ought to have a naming > convention that makes them stand out and remind you to consult the > documentation. Maybe we should mandate that their names end with '_rtfm'

Re: [Python-Dev] cpython: Fix email post-commit review comments.

2012-04-18 Thread Greg Ewing
Antoine Pitrou wrote: (and here we see why reference-stealing APIs are a nuisance: because you never know in advance whether a function will steal a reference or not, and you have to read the docs for each and every C API call you make) Fortunately, they're very rare, so you don't encounter th

Re: [Python-Dev] cpython: Fix email post-commit review comments.

2012-04-18 Thread Eric Snow
On Wed, Apr 18, 2012 at 8:21 AM, Antoine Pitrou wrote: > (and here we see why reference-stealing APIs are a nuisance: because > you never know in advance whether a function will steal a reference or > not, and you have to read the docs for each and every C API call you > make) +1 -eric _

Re: [Python-Dev] cpython: Fix email post-commit review comments.

2012-04-18 Thread Antoine Pitrou
On Wed, 18 Apr 2012 15:31:10 +0200 brian.curtin wrote: > http://hg.python.org/cpython/rev/bf23a6c215f6 > changeset: 76388:bf23a6c215f6 > parent: 76385:6762b943ee59 > user:Brian Curtin > date:Wed Apr 18 08:30:51 2012 -0500 > summary: > Fix email post-commit review comments