Crash when adding dive [was: add dive implementation pushed]
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.
More information about the subsurface