Refactored Python code base. #128
Conversation
…ile as module by accident.
| import shutil | ||
| import gzip | ||
|
|
||
| try: |
There was a problem hiding this comment.
This was needed, I think on a windows system or *nix system. Don't remember it anymore.
There was a problem hiding this comment.
imports have just been re-ordered. gzip will be still imported. Output seems the same. I would appreciate it, if you could double check it. Thanks.
There was a problem hiding this comment.
No it looks like that the try-block is gone, at least in the pull request review
There was a problem hiding this comment.
Yes it is gone. It still works. I was asking myself, why would you like to use unicode?
You can also alter the docs here: https://github.com/freeDSP/freeDSP-aurora/wiki/Create-a-Custom-Plugin
This script runs also with python3.10. Maybe you could state this in the docs as well. Since Python 2.7 is also deprecated.
Added if statement to avoid execution of this file as module by accident. See https://stackoverflow.com/questions/419163/what-does-if-name-main-do-in-python
This file has a lot of code repetition. I just getting use to it and will push more refactoring PRs to remove all the code duplication etc.