Export to GitHub

arducopter - issue #161

error in AP_GPS_NMEA.cpp


Posted on May 24, 2011 by Quick Giraffe

Arducoper NG or ACM? ACM What steps will reproduce the problem? 1. See the code 2. 3.

What is the expected output? What do you see instead?

What version of the product are you using? On what operating system? ACM 2 beta, and APM 2

Please provide any additional information below.

In libraries\AP_GPS\AP_GPS_NMEA.cpp: line 63, missing "\r\n" at the end of the sentence. line 257,258, the "new_stats" and "hdop" should be moved under the case of "_GPS_SENTENCE_GPGGA", because they exist in the GPGGA, not the GPRMC.

The modified code is attached, please check it.

Attachments

Comment #1

Posted on May 24, 2011 by Happy Kangaroo

Yeah, Im just going to commit it. None of us use NMEA. Thanks! Jason

Status: Fixed

Labels:
Type-Defect Priority-Medium