On 17.3.2014. 17:56, Guido van Rossum wrote:
That patch looks fine. I'd use a different name than 'accept_op', e.g.
'accept_future', and I'd like to delete the future again once the accept
has completed. I'd also like to see a unit test, and I'd like you to
sign the (online) PSF contributor
form:https://www.python.org/psf/contrib/contrib-form/

I am beginning to understand the D in BDFL :)

If you can, please use Rietveld (codereview.appspot.com
<http://codereview.appspot.com>) to iterate on your patch.

https://codereview.appspot.com/77150043/

Reply via email to