PEP8 radio/util/sportparse.py

Remove unused import, fix whitespace, indentation, and comments

radio/util/sport-parse.py

@@ -5,12 +5,10 @@
 
 from __future__ import division, print_function
 
-import os
 import sys
 import struct
 
 
-
 START_STOP = '\x7e'
 BYTESTUFF = '\x7d'
 STUFF_MASK = '\x20'
@@ -18,43 +16,47 @@ STUFF_MASK = '\x20'
 DATA_FRAME = 0x10
 
 
-
 lineNumber = 0
 sportDataBuff = ""
 
 quiet = False
 
 
-
 def ParseFlVSS(packet, dataId, prim, appId, data, crc):
     # if dataId != 0xa1:
     #     return
     print("packet: %s (%4d)" % (dump(packet), lineNumber), end=' ')
     cells = (data & 0xF0) >> 4
-  battnumber = data & 0xF;
+    battnumber = data & 0xF
     voltage1 = ((data & 0x000FFF00) >> 8) // 5
     voltage2 = ((data & 0xFFF00000) >> 20) // 5
     print(" FLVSS: no cells: %d, cell: %d: voltages: %0.2f %0.2f" % (cells, battnumber, voltage1 / 100., voltage2 / 100.))
 
+
 def ParseRSSI(packet, dataId, prim, appId, data, crc):
     print("packet: %s (%4d)" % (dump(packet), lineNumber), end=' ')
     print(" RSSI: %d" % (data & 0xFF))
 
+
 def ParseAdc(packet, dataId, prim, appId, data, crc):
     print("packet: %s (%4d)" % (dump(packet), lineNumber), end=' ')
     print(" A%d: %d" % (dataId - 0xf102, data & 0xFF))
 
+
 def ParseBatt(packet, dataId, prim, appId, data, crc):
     print("packet: %s (%4d)" % (dump(packet), lineNumber), end=' ')
     print(" Batt: %d" % (data & 0xFF))
 
+
 def ParseSWR(packet, dataId, prim, appId, data, crc):
     print("packet: %s (%4d)" % (dump(packet), lineNumber), end=' ')
     print(" SWR: %d" % (data & 0xFF))
 
+
 def ParseAirSpeed(packet, dataId, prim, appId, data, crc):
-  print "packet: %s (%4d)" % (dump(packet), lineNumber) ,
+    print("packet: %s (%4d)" % (dump(packet), lineNumber), end=' ')
-  print " Aspd: %.1f km/h"  % (data/10.0)    
+    print(" Aspd: %.1f km/h" % (data / 10.0))
+
 
 appIdParsers = (
     (0x0300, 0x030f, ParseFlVSS),
@@ -62,17 +64,19 @@ appIdParsers = (
     (0xf102, 0xf103, ParseAdc),
     (0xf104, 0xf104, ParseBatt),
     (0xf105, 0xf105, ParseSWR),
-  (0x0a00, 0x0a0f, ParseAirSpeed),
 )
 
+
 def dump(data, maxLen=None):
     if maxLen and len(data) > maxLen:
         data = data[:maxLen]
     return ":".join("{:02x}".format(ord(c)) for c in data)
 
+
 def CheckSportCrc(packet):
     return True
 
+
 def ParseSportPacket(packet):
     global lineNumber
     (dataId, prim, appId, data, crc) = struct.unpack('<BBHIB', packet)
@@ -89,22 +93,24 @@ def ParseSportPacket(packet):
         print("\tdataId:%02x, prim:%02x, appId:%04x, data:%08x, crc:%02x)" % (dataId, prim, appId, data, crc))
         print("\tparser for appId %02x not implemented" % appId)
 
+
 def ParsePacket(packet):
     global lineNumber
     # unstuff packet
     while True:
         pos = packet.find(BYTESTUFF)
-    if pos == -1: break
+        if pos == -1:
+            break
         # print "found stuff at %d in %s" % (pos, dump(packet, 20))
         # if we have nex char, then unstuff it
         if len(packet) > pos + 1:
-      unstuffed = ord(packet[pos]) ^ ord(STUFF_MASK);
+            unstuffed = ord(packet[pos]) ^ ord(STUFF_MASK)
             # print "unstuffed data: %02x" % unstuffed
             # buffer[pos] = chr(unstuffed)
             packet = packet[:pos] + chr(unstuffed) + packet[pos + 2:]
             # print "buff after unstuff                 %s" % dump(packet, 20)
         else:
-      #missin data, wait for more data
+            # missing data, wait for more data
             print("unstuff missing data")
             return
 
@@ -147,9 +153,6 @@ def ParseSportData(data):
             break
 
 
-
-
-
 inputFile = None
 
 if len(sys.argv) > 1:
@@ -164,9 +167,11 @@ else:
 while True:
     line = inp.readline()
     lineNumber += 1
-  if len(line) == 0: break
+    if len(line) == 0:
+        break
     line = line.strip('\r\n')
-  if len(line) == 0: continue
+    if len(line) == 0:
+        continue
     # print line
     parts = line.split(': ')
     if len(parts) < 2: