NtesEyes / pylane

An python vm injector with debug tools, based on gdb.
GNU General Public License v3.0
358 stars 34 forks source link

Built-in 'file()' was removed in Python in favor of 'open()' #4

Closed cclauss closed 5 years ago

cclauss commented 5 years ago

open() will work as expected in both Python 2 and Python 3.

flake8 testing of https://github.com/NtesEyes/pylane on Python 3.7.1

$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics

./pylane/shell/remote_shell.py:154:13: F821 undefined name 'file'
            file('/tmp/pmx-shell-log', 'w').write("%s\n%s" % (
            ^
1     F821 undefined name 'file'
1

E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc. These 5 are different from most other flake8 issues which are merely "style violations" -- useful for readability but they do not effect runtime safety.