[Python-Dev] Re: [Python-checkins] python/dist/src/Objectstupleobject.c, 2.86, 2.87 (original) (raw)

Tim Peters tim.one at comcast.net
Tue Mar 23 11:06:24 EST 2004


 arigo> memset() hunt continuing.  This is a net win.

[Skip]

Maybe we should cast this as a PyMEMSET macro to make it more obvious what's been done?

An inline memset loop is quite clear on its own, so it's fine by me if inlined-for-speed memset loops are simply commented as to why they're being done inline. A comment at the site is evidence that someone actually thought about it, and maybe even tried it both ways; using a macro without comment might well be due to monkey-see monkey-do, or inappropriate copy-&-paste. IOW, I think defining a macro would make it less obvious. Comments require more effort, and in these cases that's good.



More information about the Python-Dev mailing list