Alexander Belopolsky <belopol...@users.sourceforge.net> added the comment:
What is the rationale for swallowing all socket exceptions except "Connection reset by peer" in __exit__? In any case, it is better to use errno.ECONNRESET instead of literal 54. Note that SMTP.quit() calls SMTP.close(), so in the normal termination case, close will be called twice. This is not a real problem since SMTP.close() is a noop on a closed SMTP object, but it does not look right. The double call to close() also makes error path harder to analyze. It appears that if a socket error is raised in the first call to close, it gets caught only to be raised again in the second call (assuming a persistent error). ---------- nosy: +belopolsky _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue4972> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com