Author vstinner
Recipients josh.r, jtaylor, neologix, njs, pitrou, skrah, vstinner
Date 2014-04-29.20:59:51
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1398805194.98.0.957786844127.issue21233@psf.upfronthosting.co.za>
In-reply-to
Content
Patch version 6:

- I renamed "int zero" parameter to "int use_calloc" and move the new parameter at the first position to avoid confusion with nelem. For example, _PyObject_Alloc(ctx, 1, nbytes, 0) becomes _PyObject_Alloc(0, ctx, 1, nbytes). It also more logical to put it in the first position. In bytesobject.c, I leaved it at the parameter at the end since its meaning is different (fill bytes with zero or not) IMO.

- I removed my hack (premature optimization) "assert(nelem == 1); ... malloc(elsize);" and replaced it with a less surprising "... malloc(nelem * elsize);"

Stefan & Charles-François: I hope that the patch looks better to you.
History
Date User Action Args
2014-04-29 20:59:55vstinnersetrecipients: + vstinner, pitrou, njs, skrah, neologix, jtaylor, josh.r
2014-04-29 20:59:54vstinnersetmessageid: <1398805194.98.0.957786844127.issue21233@psf.upfronthosting.co.za>
2014-04-29 20:59:54vstinnerlinkissue21233 messages
2014-04-29 20:59:54vstinnercreate