added a terminal reset at end of assisirun

assisi / assisipy

Python API for the ASSISI|bf project.

Other

0 stars 3 forks source link

added a terminal reset at end of assisirun #98

Closed rmm-fcul closed 6 years ago

rmm-fcul commented 6 years ago

resolves #49
stty sane restores a useable terminal
the finally block is approx equivalent to using a context manager, without the need to use more dependencies

rmm-fcul commented 6 years ago

see https://stackoverflow.com/a/26096914 for discussion on try/finally vs context managers. Since a lot of the development work is in python >=3.4, and assisipy supports 2.7, I think this solution is the cleaner/minimal impact choice.