new Uemis downloader

Dirk Hohndel dirk at hohndel.org
Sat Dec 1 10:52:11 PST 2012


OK, I think I know what broke. As I feared, the 'obvious cleanup' of 
the parser.
Dang.
Will fix this afternoon and push an update.

(Very frustrating... I tested this quite a bit and then disliked a few 
lines of code when I reviewed the patches before pushing... But as I 
said at the airport I didn't test it again as they were 'obviously 
correct'. Yeah right.)

/D



On December 1, 2012 10:25:11 AM Linus Torvalds 
<torvalds at linux-foundation.org> wrote:
> On Sat, Dec 1, 2012 at 10:13 AM, Linus Torvalds
> <torvalds at linux-foundation.org> wrote:
> >
> > I'll see if I can find any obvious reason why.
>
> Dirk, adding some debugging I see
>
> Uemis 0
>   bp = {{{0000000000000000000000000000000000000... (0x7f3c140024ad
> 0x7f3c1400269d)
>   parsing tag
> End dive 0 0
>
> (That's just hooing into "uemis_start_dive()" to show the deviceid
> (0), then the second line is showing the bp buffer (and hex values of
> pointer and end), the third line shows what tag it's trying to parse
> (empty string), and the fourth dive is just before it decides to not
> even record the empty data.
>
> So something completely broken.
>
>                 Linus
> _______________________________________________
> subsurface mailing list
> subsurface at hohndel.org
> http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface




More information about the subsurface mailing list