From e890fe8e3a79a6cd9b70b452ae18ac30299ca8a4 Mon Sep 17 00:00:00 2001 From: Bryce Salmi Date: Sun, 12 Mar 2017 23:54:46 -0700 Subject: [PATCH] Update return statement Much more simple method of only having data in body be joined by commas and tacking that onto a packet type indicator to avoid having to remove a comma later. This was suggested in comments on #27. --- aprslib/packets/telemetry.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/aprslib/packets/telemetry.py b/aprslib/packets/telemetry.py index ff519b9..ca2fb38 100644 --- a/aprslib/packets/telemetry.py +++ b/aprslib/packets/telemetry.py @@ -12,7 +12,6 @@ class TelemetryReport(APRSPacket): tempio = ''.join(self.telemetry['vals'][5]) body = [ - 'T#', # packet type str(self.telemetry['seq']).zfill(3), str(self.telemetry['vals'][0]).zfill(3), str(self.telemetry['vals'][1]).zfill(3), @@ -22,7 +21,5 @@ class TelemetryReport(APRSPacket): str(tempio), self.comment, ] - tmpbody = ",".join(body) - - # remove static but erroneous comma between T# and sequenceno - return tmpbody[:2] + tmpbody[3:] + # Add packet type to body joined by commas + return 'T#' + ",".join(body)