Skip to content

Commit 443bd8d

Browse files
committed
pylint fixes
1 parent dcbf339 commit 443bd8d

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

adafruit_trellis.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
0x03, 0x10, 0x30, 0x21,
7373
0x13, 0x12, 0x11, 0x31)
7474
# pylint: enable=bad-whitespace, invalid-name
75-
75+
# pylint: disable=missing-docstring, protected-access
7676
class TrellisLEDs():
7777
def __init__(self, trellis_obj):
7878
self._parent = trellis_obj
@@ -84,16 +84,16 @@ def __getitem__(self, x):
8484
mask = 1 << (ledLUT[x % 16] & 0x0f)
8585
return bool(((self._parent._led_buffer[x // 16][led * 2] | \
8686
self._parent._led_buffer[x // 16][(led * 2) + 1] << 8) & mask) > 0)
87-
87+
8888
def __setitem__(self, x, value):
8989
if 0 < x >= self._parent._num_leds:
9090
raise ValueError(('LED number must be between 0 -', self._parent._num_leds - 1))
9191
led = ledLUT[x % 16] >> 4
9292
mask = 1 << (ledLUT[x % 16] & 0x0f)
93-
if value == True:
93+
if value:
9494
self._parent._led_buffer[x // 16][led * 2] |= mask
9595
self._parent._led_buffer[x // 16][(led * 2) + 1] |= mask >> 8
96-
elif value == False:
96+
elif not value:
9797
self._parent._led_buffer[x // 16][led * 2] &= ~mask
9898
self._parent._led_buffer[x // 16][(led * 2) + 1] &= ~mask >> 8
9999
else:
@@ -104,11 +104,12 @@ def __setitem__(self, x, value):
104104

105105
def fill(self, on):
106106
fill = 0xff if on else 0x00
107-
for d in range(len(self._parent._i2c_devices)):
107+
for buff in range(len(self._parent._i2c_devices)):
108108
for i in range(16):
109-
self._parent._led_buffer[d][i] = fill
109+
self._parent._led_buffer[buff][i] = fill
110110
if self._parent._auto_show:
111111
self._parent.show()
112+
# pylint: enable=missing-docstring, protected-access
112113

113114
class Trellis():
114115
"""

0 commit comments

Comments
 (0)