Skip to content

Commit 144bb07

Browse files
authored
Merge pull request #17 from margaret/consistent_get_set
(#9) change __getitem__ to use self._n instead of self._buf // 4
2 parents f9e5675 + befd88b commit 144bb07

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

adafruit_dotstar.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ def _set_item(self, index, value):
145145

146146
def __setitem__(self, index, val):
147147
if isinstance(index, slice):
148-
start, stop, step = index.indices(len(self))
148+
start, stop, step = index.indices(self._n)
149149
length = stop - start
150150
if step != 0:
151151
length = math.ceil(length / step)
@@ -162,7 +162,7 @@ def __setitem__(self, index, val):
162162
def __getitem__(self, index):
163163
if isinstance(index, slice):
164164
out = []
165-
for in_i in range(*index.indices(len(self._buf) // 4)):
165+
for in_i in range(*index.indices(self._n)):
166166
out.append(
167167
tuple(self._buf[in_i * 4 + (3 - i) + self.start_header_size] for i in range(3)))
168168
return out

0 commit comments

Comments
 (0)