Subsurface-mobile crash on deleting dive from dive list

[Rick Walsh]

On 6 August 2017 at 12:55, Rick Walsh <rickmwalsh at gmail.com> wrote:

>
>
> On 4 August 2017 at 14:11, Dirk Hohndel <dirk at hohndel.org> wrote:
>
>>
>> > On Aug 3, 2017, at 3:49 PM, Rick Walsh <rickmwalsh at gmail.com> wrote:
>> >
>> > Hi,
>> >
>> > This is not a new bug, and has been reported before, but I think it's
>> one that should be eliminated before the next release.  When deleting a
>> dive from the dive list (select the dive from the list but hold down until
>> the trash icon appears, then select the trash icon), the app will crash...
>> sometimes.  But sometimes it works.
>>
>> I have spent many hours on this. Yes, we either need to fix this or
>> remove the feature.
>>
> I agree, I meant to say we should remove the feature if we can't fix it.
>
>> This won't prevent another beta (it's not a regression, it's broken in
>> the current beta), but it will prevent the actual release of 2.0.
>
>

> My suspicion is that somehow our different models get out of sync. But
>> it's really, really hard to debug this :-(
>>
>
> Indeed it is
>
Unfortunately I don't understand how or why the models get out of sync, but
I just made a one-line pull request. And this time, in my testing at least,
the bug has been fixed.  Please test more.

Cheers,

Rick
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20170810/0bcf7a25/attachment.html>