[python] упростить конструкцию

desired_type = "application"
applic_parts = filter(lambda p: desired_type == p.get_content_maintype(), msg.walk())

if not applic_parts:
    exit_on_err("attachment not found")

Или

if "application" not in map(lambda p: p.get_content_maintype(), msg.walk()):
    exit_on_err("attachment not found")

Лучше не используй функциональный подход Кондора, его не рекомендует сам BDFL.

Питоновский подход такой:

for part in msg.walk():
    if part.get_content_maintype() == 'application':
        break
else:
    exit_on_err("attachment not found")

Не ставь лишние скобки вокруг if (читать PEP8 до просветления).
Также читать официальные доки по поводу конструкций языка:

http://docs.python.org/ref/for.html
http://docs.python.org/tut/node6.html#SECTION006400000000000000000

kondor,спасибо, а какой способ принято использовать в публичном коде?

Тут есть нюанс. Мне на самом деле нужно или найти аттачмент или вывести сообщение что его нет. А просто проверить есть ли он можно и так:

if not filter(lambda p: p.get_content_maintype() == "application", msg.walk()): print "no attachment"

Читабельность, правда, плохая.

Лучше послушай анонимуса выше — он общепринятый способ написал.

спасибо, я и забыл что такое есть.

а вот про скобки вокруг if в pep8 ни слова.