Coverity cleanups

Salvador CuƱat salvador.cunat at gmail.com
Sat Oct 3 11:18:25 PDT 2015


Hi Dirk.

El 03/10/2015 16:08, "Dirk Hohndel" <dirk at hohndel.org> escribiĆ³:
>
>
> I ignored this last time around (bad me) but this time I tried to clean up
> the datatrac.c file which ignored all return values from fread and fseek.
> The code now in master should be a lot more robust when given bogus data
> files - but I would love it if Salvador could check that this still
> actually works.

It works fine for me,  even in faked .log files.

>It would be good if we had a test case for this, too.
>

Do you mean automated test using QTEST macro ?  I'll take a look.
Remember there is Example.log in Dives directory for testing pourposes
(perhaps should tweak it to include some fails).

Regards.

Salva.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20151003/2ea2dc83/attachment.html>


More information about the subsurface mailing list