Hi,

> On 17. Dec 2017, at 14:19, Berthold Stoeger <[email protected]> 
> wrote:
> 
> There is another warning concerning entry in l.807. To make the warning go
> away, you could add a
>  if (pi->nr <= 0)
>    return;
> block before the loop. Even if pi->nr can never be <= 0, this seems like a
> good idea for robustness' sake.

isn’t that what

assert(pi->nr >= 0);

is for?

Best
Robert

Attachment: signature.asc
Description: Message signed with OpenPGP

_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to