<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 4, 2013 at 3:36 PM, Nick Coghlan <span dir="ltr"><<a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><p dir="ltr">
On 5 Nov 2013 08:49, "Larry Hastings" <<a href="mailto:larry@hastings.org" target="_blank">larry@hastings.org</a>> wrote:<br>
> When Clinic generates a function, it knows what kind of callable it represents, and it names the first argument (the "PyObject *") accordingly:<br>
> module-level function ("self"),<br>
> method ("self"),<br>
> class method ("cls"), or<br>
> class static ("null").<br>
> I now boldly propose that for the first one, the module-level function, the PyObject * parameter should be named "module".  The object passed in is the module object, it's not a "self" in any conventional sense of the word.<br>



><br>
> This would enhance readability, as I assert the name "self" there is confusing.  The argument is rarely used on module-level functions, and very little code is converted right now using Clinic anyway.  I therefore assert this change would break very little code, and the code that did get broken by this change could be fixed as part of the process of converting it to Clinic.<br>



><br>
> But now would be the time to make this change, before doing the big push to convert to Clinic.  (A couple of weeks ago would have been even better...!)<br>
><br>
> +1?  -1?</p>
</div></div><p dir="ltr">+1 from me, as they're not really methods of the module instance (i.e. dynamically bound when retrieved from the module), even though they behave a little like they are.</p>
<p dir="ltr">Instead of relying on the descriptor protocol (which doesn't trigger because module level functions are stored in an instance namespace), we play games when creating the Python level callables during module creation in order to prebind the module object<br>

</p></blockquote><div>+1 from me too. This finally fixes a silly mistake I made well over two decades ago! <br></div></div><br>-- <br>--Guido van Rossum (<a href="http://python.org/~guido">python.org/~guido</a>)
</div></div>