Milan Oberkirch added the comment:

Thanks for the review and improvements!

I fixed the warnings (didn't see them with running tests as described in the 
dev guide; I may improve that later), updated the docs to RFC 5321 (issue 19679 
is referring to RFC 3463 which is not required/updated/obsoleted by RFC 5321), 
and fixed the _print_message method of the DebuggingServer.

Testing DebuggingServer seems difficult because it prints to stdout so I'm just 
checking that no exceptions are raised.

I'm aware of the maximum command length thing but would fix it in issue 21783 
(because it becomes a bug when dealing with that).

----------
Added file: http://bugs.python.org/file36149/issue21725v5.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue21725>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to