lint: Clean up E741 PEP-8 rule.

tools/lint-all

@@ -89,7 +89,6 @@ def check_pep8(files):
         #
         'E123', 'E126', 'E226', 'E241', 'E261', 'E302',
         'E401', 'E501', 'E702', 'E711', 'E712', 'E713', 'E714',
-        'E741',
 
         #
         # Each of these rules are ignored for the explained reason.

zerver/lib/ccache.py

@@ -37,14 +37,14 @@ import six
 # there is already an ASN.1 implementation, but in the interest of
 # limiting MIT Kerberos's exposure to malformed ccaches, encode it
 # ourselves. To that end, here's the laziest DER encoder ever.
-def der_encode_length(l):
+def der_encode_length(length):
     # type: (int) -> str
-    if l <= 127:
-        return chr(l)
+    if length <= 127:
+        return chr(length)
     out = ""
-    while l > 0:
-        out = chr(l & 0xff) + out
-        l >>= 8
+    while length > 0:
+        out = chr(length & 0xff) + out
+        length >>= 8
     out = chr(len(out) | 0x80) + out
     return out
 

zerver/tests/test_type_debug.py

@@ -14,9 +14,9 @@ def add(x=0, y=0):
     # type: (Any, Any) -> Any
     return x + y
 
-def to_dict(l=[]):
+def to_dict(v=[]):
     # type: (Iterable[Tuple[Any, Any]]) -> Dict[Any, Any]
-    return dict(l)
+    return dict(v)
 
 class TypesPrintTest(TestCase):
 
@@ -120,9 +120,9 @@ class TypesPrintTest(TestCase):
         class A(dict):
             pass
 
-        def to_A(l=[]):
+        def to_A(v=[]):
             # type: (Iterable[Tuple[Any, Any]]) -> A
-            return A(l)
+            return A(v)
 
         self.check_signature("to_A() -> A([])", A(()), to_A)
         self.check_signature("to_A([(int, str)]) -> A([(int, str)])",