<div dir="ltr">The discussion has drifted towards improving OrderedDict (something I certainly approve), but the semantic question is still there: why should BoundArguments.arguments be ordered by the parameter order? For example, the recipe just below in the docs, for filling in the remaining default arguments, breaks that ordering, without even mentioning that.<br><br>Antony<br></div><div class="gmail_extra"><br><div class="gmail_quote">2014-12-17 16:37 GMT-08:00 Nick Coghlan <span dir="ltr"><<a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>></span>:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 18 December 2014 at 07:26, Eric Snow <<a href="mailto:ericsnowcurrently@gmail.com">ericsnowcurrently@gmail.com</a>> wrote:<br>
>> - does it have to inherit from dict? that looks like a potential can of<br>
>> worms<br>
><br>
> One of the constraints here is to make the C implementation of<br>
> OrderedDict match the API of the Python type exactly and the<br>
> underlying implementation as closely as reasonable. The Python type<br>
> subclasses dict so the C implementation does as well.<br>
<br>
</span>This kind of design rationale is potentially useful to include inline<br>
as a block comment. The kinds of questions reviewers have are often<br>
going to be the same kinds of questions future maintainers have, and<br>
"why" data regarding core architectural decisions is less likely to go<br>
out of date as future maintainers make changes.<br>
<span class=""><br>
> FWIW (and not directed to Antoine specifically), the implementation up<br>
> for review may not be ideal, but it exists and is complete. :)<br>
> Barring any substantial concerns during review (and the points Antoine<br>
> has brought up), I would rather the patch landed than wait<br>
> indefinitely for a more ideal implementation. That could happen<br>
> afterward, though I'm fairly confident in the correctness and<br>
> efficiency of the implementation.<br>
<br>
</span>Aye, a definite +1 for "better" over "perfect".<br>
<br>
Cheers,<br>
Nick.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Nick Coghlan | <a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a> | Brisbane, Australia<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Python-ideas mailing list<br>
<a href="mailto:Python-ideas@python.org">Python-ideas@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-ideas" target="_blank">https://mail.python.org/mailman/listinfo/python-ideas</a><br>
Code of Conduct: <a href="http://python.org/psf/codeofconduct/" target="_blank">http://python.org/psf/codeofconduct/</a><br>
</div></div></blockquote></div></div>