(original) (raw)

What is wrong with "except:" in this specific case?


Victor

Le mardi 26 mai 2015, Yury Selivanov <yselivanov.ml@gmail.com> a écrit :




On 2015-05-26 12:26 AM, Terry Reedy wrote:

\+ try:
\+ encoding, lines = detect\_encoding(buffer.readline)
\+ buffer.seek(0)
\+ text = TextIOWrapper(buffer, encoding, line\_buffering=True)
\+ text.mode = 'r'
\+ return text
\+ except:
\+ buffer.close()
\+ raise

Please do not add bare 'except:'. If you mean 'except BaseException:', say so.

try..finally would be even better.

Yury
\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_
Python-Dev mailing list
Python-Dev@python.org
https://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: https://mail.python.org/mailman/options/python-dev/victor.stinner%40gmail.com