Cleanup

[Kriechi]

• cleanup code with autopep8
• cleanup code with autoflake
• add pep8 autoformat check to travis

run the following command:
$ autopep8 -i -r -a -a .
$ autoflake -i -r --remove-all-unused-imports --remove-unused-variables .

[coveralls]

Coverage decreased (-0.21%) to 99.37% when pulling 7be95b6 on Kriechi:cleanup into f7b75ba on mitmproxy:master.

[coveralls]

Coverage decreased (-0.21%) to 99.37% when pulling a11f2a6 on Kriechi:cleanup into f7b75ba on mitmproxy:master.

[coveralls]

Coverage decreased (-0.21%) to 99.37% when pulling 8037830 on Kriechi:cleanup into f7b75ba on mitmproxy:master.

[mhils]

This shouldn't be neccessary if you install mitmproxy using the instructions in https://github.com/mitmproxy/mitmproxy#hacking

[Kriechi]

It is - somehow Travis creates this directory - and then the git-status check fails.
See https://travis-ci.org/mitmproxy/netlib/jobs/64226073#L199

Maybe it would be safer to remove the new Travis checks? What do you think?

[cortesi]

Lets merge this and see. There are a few things that autopep8 does that I don't like (blank lines before class comments, for example), which we can exempt using the --ignore flag. I also think we should extract the autoflake and autopep8 commands into a script that we can run, add to a commit hook, configure our editors to run, etc. that makes sure what we do is consistent with what Travis does.

[mhils]

What are your opinions on https://github.com/google/yapf? Worth a shot?

[cortesi]

Hmm... looks interesting. Let me play with it. I would love to have something that you can trust to just apply a totally consistent style, a la gofmt.