Message 297444 - 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.

In-reply-to
Author	vstinner
Recipients	belopolsky, martin.panter, musically_ut, r.david.murray, serhiy.storchaka, vstinner
Date	2017年06月30日.20:59:31
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1498856371.62.0.314934273021.issue30302@psf.upfronthosting.co.za>

Content
Serhiy: "The C code looks cumbersome. It could be made a little simpler if accumulate arguments into a string rather than a list." Utkarsh's first version used C code. I proposed to use a PyList and PyUnicodeObject strings to not have to compute the size of the char[] buffers. The annoying part is to handle the ", " separator. I also had bad experiences with handling char* strings. It's so easy to make mistakes :-( IMHO the current C code using Python objects is not that complex, but I don't really care of the implementation as soon as it works :-)

Content

Serhiy: "The C code looks cumbersome. It could be made a little simpler if accumulate arguments into a string rather than a list."
Utkarsh's first version used C code. I proposed to use a PyList and PyUnicodeObject strings to not have to compute the size of the char[] buffers.
The annoying part is to handle the ", " separator. I also had bad experiences with handling char* strings. It's so easy to make mistakes :-(
IMHO the current C code using Python objects is not that complex, but I don't really care of the implementation as soon as it works :-)

History
Date	User	Action	Args
2017年06月30日 20:59:31	vstinner	set	recipients: + vstinner, belopolsky, r.david.murray, martin.panter, serhiy.storchaka, musically_ut
2017年06月30日 20:59:31	vstinner	set	messageid: <1498856371.62.0.314934273021.issue30302@psf.upfronthosting.co.za>
2017年06月30日 20:59:31	vstinner	link	issue30302 messages
2017年06月30日 20:59:31	vstinner	create

homepage