Crash when adding dive [was: add dive implementation pushed]

Dirk Hohndel dirk at hohndel.org
Tue Sep 24 12:35:15 UTC 2013


On Tue, 2013-09-24 at 11:55 -0700, Dirk Hohndel wrote:
> On Tue, 2013-09-24 at 20:46 +0200, Henrik Brautaset Aronsen wrote:
> > Dirk Hohndel wrote: 
> > > On Sun, 2013-09-22 at 23:01 +0200, Henrik Brautaset Aronsen wrote:
> > > > Here's another one:
> > > > 
> > > > 1) Add a dive
> > > > 2) Delete it
> > > > 3) The dive details are still there, so try editing and press OK.
> > > > *crash*
> > > 
> > > Fixed in the latest master.
> > 
> > Not quite:
> > 
> > 1) Add a dive
> > 2) Press cancel
> > 3) Observe that the log fields are still editable
> > 4) Go to equipment, press (+)
> > 
> > Now you either get at crash, or equipment with fun values from the
> > memory.
> 
> Heh. Another path to plug. But it should be easy now that I figured it
> out once (famous last words)

Yeah, was a little harder because of another hidden subtle bug (we
didn't correctly deal with the selection when deleting the single dive).
But should be fixed in master, now.

Thanks

/D



More information about the subsurface mailing list