Update gps.c

Now properly uses checksum before applying data in NMEA which corrects sign dropping in Latitude and Longitude
I don't have any plans to do changes in GPS.c - so no problem.


On Thu, Jul 17, 2014 at 12:43 PM, norem <notifications@github.com> wrote:

> In src/gps.c:
>
> > @@ -940,68 +944,111 @@ static bool gpsNewFrameNMEA(char c)
> >      uint8_t frameOK = 0;
> >      static uint8_t param = 0, offset = 0, parity = 0;
> >      static char string[15];
> > -    static uint8_t checksum_param, frame = 0;
> > -
> > -    if (c == '$') {
> > +    static uint8_t checksum_param, gps_frame = NO_FRAME;
> > +    static int32_t GPS_temp[2];
> > +    static uint8_t GPS_numSat_temp;
> > +    static uint16_t GPS_altitude_temp, GPS_speed_temp;
> > +    static uint16_t GPS_ground_course_temp;
>
> Totally agree, I'm driving cross country, using my cellphone to do the
> edit ;) would you rather I hold off ?, that part is gross and will be
> addressed, I'm just concerned that "correct working" code get in here
> before too many other changes get ahead of it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/multiwii/baseflight/pull/139/files#r15039985>.
>