[Python-Dev] Why not using "except: (...) raise" to cleanup on error?
Victor Stinner
vstinner at redhat.com
Mon Jun 4 12:11:55 EDT 2018
More information about the Python-Dev mailing list
Mon Jun 4 12:11:55 EDT 2018
- Previous message (by thread): [Python-Dev] Microsoft to acquire GitHub for $7.5 billion
- Next message (by thread): [Python-Dev] Why not using "except: (...) raise" to cleanup on error?
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi, I just read a recent bugfix in asyncio: https://github.com/python/cpython/commit/9602643120a509858d0bee4215d7f150e6125468 + try: + await waiter + except Exception: + transport.close() + raise Why only catching "except Exception:"? Why not also catching KeyboardInterrupt or MemoryError? Is it a special rule for asyncio, or a general policy in Python stdlib? For me, it's fine to catch any exception using "except:" if the block contains "raise", typical pattern to cleanup a resource in case of error. Otherwise, there is a risk of leaking open file or not flushing data on disk, for example. Victor
- Previous message (by thread): [Python-Dev] Microsoft to acquire GitHub for $7.5 billion
- Next message (by thread): [Python-Dev] Why not using "except: (...) raise" to cleanup on error?
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
More information about the Python-Dev mailing list