commit cca1b3c9854fc04ad3c4ff13a22d1ff9c632b5fe Author: Damian Johnson atagar@torproject.org Date: Sun Jan 6 19:40:43 2013 -0800

Conforming to E711 (None checks by identity)

E711 is actually something that we *were* trying to comply with (as are several other PEP8 warnings we're getting, actually). Fixing the one occurance. --- stem/connection.py | 2 +- test/check_whitespace.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/stem/connection.py b/stem/connection.py index 3ca4462..e4760ef 100644 --- a/stem/connection.py +++ b/stem/connection.py @@ -183,7 +183,7 @@ def _connect(control_socket, password, chroot_path, controller): try: authenticate(control_socket, password, chroot_path)

- if controller == None: + if controller is None: return control_socket else: return controller(control_socket) diff --git a/test/check_whitespace.py b/test/check_whitespace.py index f9008f5..7b7a181 100644 --- a/test/check_whitespace.py +++ b/test/check_whitespace.py @@ -47,7 +47,7 @@ def pep8_issues(base_path = DEFAULT_TARGET): # We're gonna trim these down by cateogry but include the pep8 checks to # prevent regression.

- ignored_issues = "E111,E121,W293,E501,E302,E701,E251,E261,W391,E127,E241,E128,E226,E231,E202,E201,E203,E124,E211,E222,E225,E221,E126,E262,E271,E502,E303,E711" + ignored_issues = "E111,E121,W293,E501,E302,E701,E251,E261,W391,E127,E241,E128,E226,E231,E202,E201,E203,E124,E211,E222,E225,E221,E126,E262,E271,E502,E303"

issues = {} pep8_output = system.call("pep8 --ignore %s %s" % (ignored_issues, base_path))