laptopc700 / arducopter

Automatically exported from code.google.com/p/arducopter
0 stars 0 forks source link

error in AP_GPS_NMEA.cpp #161

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
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.

Original issue reported on code.google.com by hazy...@gmail.com on 24 May 2011 at 5:03

Attachments:

GoogleCodeExporter commented 9 years ago
Yeah, Im just going to commit it. None of us use NMEA. 
Thanks!
Jason

Original comment by jasonshort on 24 May 2011 at 5:15