[Python-Dev] Re: [Python-checkins]
python/dist/src/Objectstupleobject.c, 2.86, 2.87
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 _Py_MEMSET 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