<br><br><div class="gmail_quote">On Sun, Aug 14, 2011 at 19:36, Benjamin Peterson <span dir="ltr"><<a href="mailto:benjamin@python.org">benjamin@python.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
2011/8/14 Brett Cannon <<a href="mailto:brett@python.org">brett@python.org</a>>:<br>
<div class="im">><br>
><br>
> On Thu, Aug 11, 2011 at 00:02, Antoine Pitrou <<a href="mailto:solipsis@pitrou.net">solipsis@pitrou.net</a>> wrote:<br>
>><br>
>> Le Thu, 11 Aug 2011 03:34:37 +0200,<br>
>> brian.curtin <<a href="mailto:python-checkins@python.org">python-checkins@python.org</a>> a écrit :<br>
>> > <a href="http://hg.python.org/cpython/rev/77a65b078852" target="_blank">http://hg.python.org/cpython/rev/77a65b078852</a><br>
>> > changeset: 71809:77a65b078852<br>
>> > parent: 71803:1b4fae183da3<br>
>> > user: Brian Curtin <<a href="mailto:brian@python.org">brian@python.org</a>><br>
>> > date: Wed Aug 10 20:05:21 2011 -0500<br>
>> > summary:<br>
>> > Add Py_RETURN_NOTIMPLEMENTED macro. Fixes #12724.<br>
>><br>
>><br>
>> It would sound more useful to have a generic Py_RETURN() macro rather than<br>
>> some specific forms for each and every common object.<br>
><br>
> Since the macro is rather generic, sure, but the name should probably be<br>
> better since it doesn't necessarily convene the fact that a INCREF has<br>
> occurred. So maybe Py_INCREF_RETURN()?<br>
<br>
</div>That nearly nullifies the space saving. I think that fact that it's a<br>
macro at all conveys that it does something else aside from "return<br>
x;".<br></blockquote><div><br></div><div>This is C code; space savings went out the window along with gc a long time ago.</div><div><br></div><div>Yes, being a macro helps differentiate semantics that a longer name is probably not needed.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
<br>
--<br>
Regards,<br>
<font color="#888888">Benjamin<br>
</font></blockquote></div><br>