[PATCH] Identify below floor event type properly.

Jef Driesen jef at libdivecomputer.org
Mon Aug 5 04:29:51 UTC 2013


On 2013-07-27 17:30, Dirk Hohndel wrote:
> On Jul 27, 2013, at 2:29 AM, Michael Andreen wrote:
> On Saturday 27 July 2013 11:20:25 Miika Turkia wrote:
> However, the "below floor" event should not
> hog up the "unknown" event. The unknown is used for all the events that 
> we
> do not have a mapping for. This includes also other dive computers than
> Suunto.
> 
> That was what I was trying to a discussion about with my first mail. 
> Currently
> type 16 is only used for this specific event in libdivecomputer, so 
> renaming
> it means old dives will have the right type id, making it easy to 
> update the
> text later, and subsurface can be updated independently of 
> libdivecomputer.
> 
> I guess it's mainly Jef's call. I sent the companion patch to the
> libdivecomputer list.
> 
> Jef is traveling. I'll wait for him to decide how he wants this solved
> before applying the patch.

I'm fine with renaming the unknown event.

I'll continue the discussion as a reply to your patch on the 
libdivecomputer mailinglist (with the subsurface list in CC).

Jef


More information about the subsurface mailing list