Fix merging of weight systems

Dirk Hohndel dirk at hohndel.org
Mon Oct 29 11:46:26 PDT 2012


Linus Torvalds <torvalds at linux-foundation.org> writes:

> On Mon, Oct 29, 2012 at 11:36 AM, Dirk Hohndel <dirk at hohndel.org> wrote:
>>
>> Turns out that on the Uemis you can add weight information on the dive
>> computer and we can indeed parse that from Subsurface. So there it is
>> entirely possible (even likely) that you have manually entered weight
>> info with the correct weight description, yet you get weight info from
>> the computer on a new download with description "unknown". I think the
>> right thing to do is:
>>
>> if both have weight info
>>   if the total weight is the same
>>     keep the one that doesn't have "unknown" as description
>>
>> All other cases: accumulate all of the weight info from both "dives"
>
> Right now, it prefers the older version if there's any weight
> information there. So that should work too for the Uemis import case.
> But yeah, we could have some "fill in from both dives" model too.

Ok, I'll push it as is and we can fine-tune it if necessary once I play
with it.

/D


More information about the subsurface mailing list