summaryrefslogtreecommitdiff
path: root/isgps.c
Commit message (Collapse)AuthorAgeFilesLines
* Retire splint from our set of static analyzers.Eric S. Raymond2015-03-301-9/+1
| | | | | | | | | | | | | | | | | | | The proximate cause was that we've been seing emission of error messages that were randomly and disturbingly variable across different environments - notably Raspbian and Gentoo splint gave nontrivially different results than Ubuntu 14.10 splint. And this was *not* due to Ubuntu patches! A pristine splint built from the 3.1.2 tarball on Ubuntu didn't match the Raspbian and Gentoo results either. But this has been coming for a while. Easy access to more modern static analyzers such as coverity, scan-build and cppcheck has been decreasing the utility of splint, which is unmaintained and somewhat buggy and not easy to use. Only file not cleaned is ppsthread.c, because Gary has been working on it during this cleanup. All regression tests pass. PPS observed live on GR601-W.
* gpsd-report() -> gpsd_log()Eric S. Raymond2015-03-071-28/+28
| | | | | | | | | | | | | | | | This change is done so we can add a "log" hook to the pps_thread_t structure (this is not done yet) and harmonize with the name of the outer logging function. If that name had been left as gpsd_report() there would have been scope for bad confusion with the report_hook member. Also, remove two stray duplicative printf calls from the NMEA2000 driver (drivers shouldn't have printfs!) and fix one typo. This is a step towards factoring out ntplib. For that to happen, the PPS thread code needs to be decoupled from the core session structure. No logic changes. Object compatibility preserved. All regression tests pass.
* Now that the transition is done we can restore the gpsd_report name.Eric S. Raymond2014-08-271-12/+12
| | | | All regression tests pass.
* All gpsd_reportcalls are gone. Only the unused definitins are left.Eric S. Raymond2014-08-271-1/+1
| | | | | | Next, implement labeled reporting and fix up gpson to do the right thing. All regression tests pass.
* Introduce struct errout_t to encapsulate error-reporting hooks.Eric S. Raymond2014-08-271-11/+11
| | | | | | A major step towards eliminating reverse linkage. All regression tests pass.
* Eliminate gpsd_report() from isgps.c. All regression tests pass.Eric S. Raymond2014-08-271-11/+11
|
* Fix a misleading variable name. No logic changes.Eric S. Raymond2014-08-271-66/+66
| | | | All regressuin tests pass.
* Structure member name change: packet -> lexer. No logic changes.Eric S. Raymond2014-08-271-3/+3
| | | | All regression tests pass.
* Code is now static-checker clean with splint, cppcheck, and Coverity.Eric S. Raymond2013-11-191-0/+2
| | | | All regression tests pass. PPS is live.
* Change gpsd_report to no longer use a global. All regression tests pass.Eric S. Raymond2013-09-291-11/+15
| | | | | | | | This is a large, ugly change. But without it we can't troubleshoot the ICP/IP-source initialization bug properly - colliding definitions of gpsd_report() were interfering with error reporting early in gpsd runs. More cleanup work remains to be done, but at least this is working.
* Nuke trailing whitespace in C source.Eric S. Raymond2012-09-071-12/+12
|
* Clean up from cppcheck warnings. All regression tests pass.Eric S. Raymond2012-05-171-2/+2
|
* Minor improvements to RTCM2 doc comments.Eric S. Raymond2012-04-171-2/+2
|
* Improve an explanatory comment.Eric S. Raymond2011-03-171-1/+1
|
* Make the length of ISGPS packets accessible.Eric S. Raymond2011-02-261-0/+2
|
* More deheader testing. All regression tests pass.Eric S. Raymond2011-01-131-0/+1
|
* More header portability auditing by deheader.Eric S. Raymond2010-12-221-4/+0
|
* Unused-header removals by deheader. All regression tests pass.Eric S. Raymond2010-12-011-4/+0
|
* XXX -> FIX-ME: Avoids false matches with autoconf cruft.Eric S. Raymond2010-05-121-1/+0
| | | | | | | So we can grep for things that need to be cleaned up without a lot of noise. All regression tests pass.
* dead code removalChris Kuethe2010-04-181-31/+0
| | | | | | 1) if this is debug, it should be in gpsd_report 2) if this is protocol IO, it should go in a json message 3) either way, we're not using this
* isgps_parityok is a macro, delete functionChris Kuethe2010-04-181-11/+3
| | | | | | rearrange the comment a little to explain why we're using a macro to do this, rather than a function. no point in keeping a commented out function.
* More GNU indent pragmas.Eric S. Raymond2010-04-141-2/+3
|
* Reindented the packet sniffer an ISGPS code. All regression tests pass.Eric S. Raymond2010-04-141-38/+52
|
* Vanish away Id and Rev $ keywords, git won't expand them to anything useful.Eric S. Raymond2010-03-131-5/+3
|
* Copyright stamps everywhere. No code changes.Eric S. Raymond2010-03-111-1/+5
|
* Include the whole config file in gpsd.h.Eric S. Raymond2009-11-201-1/+0
| | | | | | | That is, instead of sectioning out two little config defines and putting them in. This makes gpsd.h self-copntained (e.g. in case it gets installed as a library header) and means we can get rid of most inclusions of it.
* Fix some whitespace glitches found while experimenting with indent(1).Eric S. Raymond2009-11-171-1/+1
| | | | All regression tests pass.
* Guard some inclusions that splint doesn't like. Eric S. Raymond2009-09-101-0/+2
| | | | No effect on compilation. All regression tests pass.
* fix format warningChris Kuethe2009-01-061-3/+3
|
* Cosmetic changes.Eric S. Raymond2008-07-121-4/+4
|
* Note that RTCM104V3 doesn't use the ISGPS underlayer.Eric S. Raymond2008-07-121-9/+12
|
* No demand for Magnavox output yet, so we'll condition out that code...Eric S. Raymond2006-12-041-9/+8
| | | | | ...after having made sure it compiles with the current version of the ISGP layer.
* Make the code for Magnavox-format output work again...Eric S. Raymond2006-12-041-0/+32
| | | | ...though it's still not hooked up to anything in rtcmdecode().
* Improved documentation for the ISGPS layer. Eric S. Raymond2006-12-041-20/+11
| | | | | Allow this module to be compiled even if vendor binary protocol handling has been disabled.
* Split the packet-sniffer internals out of the session structure. Eric S. Raymond2006-11-301-49/+49
| | | | | | | | | | | | | This is a big, super-intrusive patch but changes no logic at all -- it's all about ripping out some of the gps_device_t structure members into a new gps_packet_t structure. Even the driver API doesn't change at all, this is all libgpsd(3) internals being rearranged. The motivation here is that we want to kill off the ad-hoc Python implementation of a packet-sniffer in gpsfake. To do that we need to be able to write a "pure" packet sniffer that uses the same C code as the daemon's but without being welded to the rest of the libgpsd(3) code. This is the first step towards that.
* Rename config.h so it can be installed as gpsd_config.h...Chris Kuethe2006-10-261-0/+2
| | | | | | ...for those who want to build their own apps linked against libgps and want the headers to work. Works on OpenBSD, tested by Jeff Francis on OS X and Linux
* Add subversion ID tags near the beginning of every file.Chris Kuethe2006-09-211-0/+1
| | | | First line where possible, second line for scripts called with #!
* Enable eveything to build when RTCM104 is configured out.Eric S. Raymond2005-12-201-0/+3
|
* splint and error-message cleanup.Eric S. Raymond2005-08-031-21/+22
|
* Parity computation and splint cleanup.Eric S. Raymond2005-07-281-1/+1
|
* Upper level of RTCM encoder works.Eric S. Raymond2005-07-281-6/+6
|
* Sepatate character from retry counters.Eric S. Raymond2005-07-261-1/+1
|
* Comment fixups.Eric S. Raymond2005-07-261-1/+1
|
* Big-endian support in rtcm.c.Eric S. Raymond2005-07-261-15/+0
|
* Support compilers that can't do anonymous unions.Eric S. Raymond2005-07-261-46/+46
|
* A large step towards automatically syncing with RTCM streams.Eric S. Raymond2005-07-261-1/+3
| | | | It partly works now! All GPS regression tests pass.
* Fix some documentation.Eric S. Raymond2005-07-251-39/+27
|
* Break isgps.c apart from rtcm.c.Eric S. Raymond2005-07-251-0/+334
View Lorry Controller Status