Weird editing dives

Linus Torvalds torvalds at linux-foundation.org
Tue Aug 21 14:31:30 PDT 2012


On Tue, Aug 21, 2012 at 2:21 PM, Dirk Hohndel <dirk at hohndel.org> wrote:
>
> This will work around your immediate bug. I need to run for an
> appointment, edit_dive_info is broken right now and needs to be fixed
> (so don't try to manually add dives right now)

I think the easy fix is to just change edit_multi_dive_info() to take
the single dive as the argument, with NULL meaning "current dive
selection".

That is actually a nicer interface anyway, and avoids the confusion
between index and dive number.

Something like the attached. Untested.

              Linus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.diff
Type: application/octet-stream
Size: 2609 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20120821/77b59fff/attachment-0001.obj>


More information about the subsurface mailing list