Patch in attachment provides a context manager for SMTP class so that it can be used with the "with" statement.

you didn't attach anything, Giampaolo.

Ouch! Here. =)

is print really necessary in the test?Also, I think it would be better to unpack the tuple in test, rather then index it.

> is print really necessary in the test?no, my mistake

A couple more minor issues:1. Double space required at the beginning of the "Here is a ..." sentence inDoc/library/smtplib.rst2. in __exit__ method: "msg" is the variable name assigned from docmd('QUIT') but "errmsg" is used when raising exception3. in test I'd rather prefer seeing stmp.noop() unpacked:    code, _ = smtp.noop()4. extra line #118 in test_smtplib5. is there any way to test SMTPResponseException?

Thanks for the contribution!  It seems like a reasonable new feature, so I'll get this landed in 3.3.  Probably should use :versionadded: instead (thanks gps).  I'll address SilentGhost's issues in the commit.

@barry I have commit privileges, but no problem if you want to commit it.

On Mar 15, 2011, at 06:40 PM, Giampaolo Rodola' wrote:>>Giampaolo Rodola' <g.rodola@gmail.com> added the comment:>>@barry I have commit privileges, but no problem if you want to commit it.@giampaolo: I'd like to do it (and am almost ready to), mostly to test out hgworkflow, but also because I have some additions which address SilentGhost'scomments.  Thanks!  (Don't worry, you'll get credit in NEWS :).

Go on, no prob.

New changeseta6a94cfb75e9 by Barry Warsaw in branch 'default':- Issue#11289: `smtp.SMTP` class becomes a context manager so it can be usedhttp://hg.python.org/cpython/rev/a6a94cfb75e9