Skip to content

Commit fd4e191

Browse files
committed
Use constants instead of magic numbers
1 parent a93c099 commit fd4e191

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

adafruit_esp32spi/adafruit_esp32spi_socket.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import time
3434
import gc
3535
from micropython import const
36+
from adafruit_esp32spi import adafruit_esp32spi
3637

3738
_the_interface = None # pylint: disable=invalid-name
3839
def set_interface(iface):
@@ -165,15 +166,14 @@ def connected(self):
165166
return True
166167
else:
167168
status = _the_interface.socket_status(self.socknum)
168-
# TODO: why is esp.<ConstantName> not defined? using magic numbers in mean time
169-
result = status not in (1,
170-
0,
171-
5,
172-
6,
173-
10,
174-
2,
175-
3,
176-
7)
169+
result = status not in (adafruit_esp32spi.SOCKET_LISTEN,
170+
adafruit_esp32spi.SOCKET_CLOSED,
171+
adafruit_esp32spi.SOCKET_FIN_WAIT_1,
172+
adafruit_esp32spi.SOCKET_FIN_WAIT_2,
173+
adafruit_esp32spi.SOCKET_TIME_WAIT,
174+
adafruit_esp32spi.SOCKET_SYN_SENT,
175+
adafruit_esp32spi.SOCKET_SYN_RCVD,
176+
adafruit_esp32spi.SOCKET_CLOSE_WAIT)
177177
if not result:
178178
self.close()
179179
self._socknum = NO_SOCKET_AVAIL

0 commit comments

Comments
 (0)