Re: [Gpsbabel-code] DG-100 GPS data logger support

[Robert L. <rob...@gm...>]

On 9/9/07, Mirko Parthey <mir...@in...> wrote:
>
> Hi,
> please consider applying the below patch to dg-100.c.
> Changelog:
> - increase RX timeout because dg100_erase() needs >20s
> - remove unused variables in dg100_getfileheaders()
>   which were moved into a local code block for debugging
> - add more debugging output to dg100_getfileheaders()
> - fix loop condition in dg100_getfileheaders()


Looks good to me.  Applied.  Thanx.



About the last item (loop condition change):
> The spec says I should continue reading headers as long as
> (numheaders!=0),
> and that's what I did. However, there was a problem when reading from a
> DG-100 with full memory - numheaders never became zero, and nextheader
> wrapped around:


Welcome to GPS vendor specs. :-)