ugps: Only convert lat and lon to negative after parsing
authorGiovani Espindola <GEspindola@advantech-bb.com>
Fri, 20 Oct 2017 12:19:02 +0000 (10:19 -0200)
committerJohn Crispin <john@phrozen.org>
Mon, 12 Feb 2018 14:39:31 +0000 (15:39 +0100)
This is a bugfix for package ugps.

Current code parses following GPRMC

$GPRMC,185834.0,A,2335.766399,S,04635.923564,W,0.0,,191017,0.0,E,A*20

as

position: -22.929440 -45.932056

with this fix it's being parsed correctly as

position: -23.596106 -46.598721

Signed-off-by: Giovani Milanez Espindola <gespindola@advantech-bb.com>
>From 092b311a3c8911ce9da3f913787b77397e88b4a8 Mon Sep 17 00:00:00 2001
From: Giovani Espindola <gespindola@advantech-bb.com>
Date: Thu, 19 Oct 2017 15:54:05 -0200
Subject: [PATCH] Only convert lat and lon to negative after parsing

nmea.c

diff --git a/nmea.c b/nmea.c
index 275f39f690e7e96215eae90e5eecf3a311a62d1a..1a3f94f5b6347a3d57a45b99cac76dcaf991434a 100644 (file)
--- a/nmea.c
+++ b/nmea.c
@@ -123,11 +123,6 @@ nmea_rmc_cb(void)
                float lat = strtof(nmea_params[3].str, NULL);
                float lon = strtof(nmea_params[5].str, NULL);
 
-               if (*nmea_params[4].str == 'S')
-                       lat *= -1.0;
-               if (*nmea_params[6].str == 'W')
-                       lon *= -1.0;
-
                degrees = floor(lat / 100.0);
                minutes = lat - (degrees * 100.0);
                lat = degrees + minutes / 60.0;
@@ -136,6 +131,11 @@ nmea_rmc_cb(void)
                minutes = lon - (degrees * 100.0);
                lon = degrees + minutes / 60.0;
 
+               if (*nmea_params[4].str == 'S')
+                       lat *= -1.0;
+               if (*nmea_params[6].str == 'W')
+                       lon *= -1.0;            
+
                snprintf(latitude, sizeof(latitude), "%f", lat);
                snprintf(longitude, sizeof(longitude), "%f", lon);