This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author berker.peksag
Recipients Claudiu.Popa, ahojnnes, berker.peksag, brian.curtin, santoso.wijaya
Date 2016-06-01.04:00:49
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1464753649.75.0.712971419611.issue7769@psf.upfronthosting.co.za>
In-reply-to
Content
Here's some quick review comments:

* xmlrpc_register_decorator_py33.patch doesn't apply cleanly anymore.

* -        serv.register_function(my_function)
  +        serv.register_function(_my_function, name='my_function')

  We should keep ``serv.register_function(my_function)`` as is and add a separate line that uses the new 'name' parameter.

* We can now change set([...]) to {...] in test_introspection1

* +            add_result, pow_result, div_result, \
  +                    myfunc_result, myfunc2_result = multicall()

  No need to use the \ character.

* The docstring of Lib/xmlrpc/server.py is already too long. It's better not to update it.

* Please use the existing code style (name = None -> name=None)

* We can make register_function act both as a decorator and as a decorator factory without changing its signature.

* We need to add a test to cover TypeError case.

* Documentation is missing. See Doc/library/xmlrpc.server.rst.

* Please add a note to Doc/whatsnew/3.6.rst.
History
Date User Action Args
2016-06-01 04:00:49berker.peksagsetrecipients: + berker.peksag, brian.curtin, ahojnnes, santoso.wijaya, Claudiu.Popa
2016-06-01 04:00:49berker.peksagsetmessageid: <1464753649.75.0.712971419611.issue7769@psf.upfronthosting.co.za>
2016-06-01 04:00:49berker.peksaglinkissue7769 messages
2016-06-01 04:00:49berker.peksagcreate