Fwd: Re: Dive Site Management - request for feedback

[Dirk Hohndel]

On Wed, May 27, 2015 at 06:13:49AM -0700, Dirk Hohndel wrote:
> > 
> >    2) I create a new dive. The original blue message that said
> >    something like "Move to the divespot and double-click the site" is
> >    not there any more. If I double-click the dive spot on marble, then
> >    I get a segmentation fault.
> 
> Maybe not ideal.

Fixed the crash. This is now a reasonable workflow to get a new site for
the dive

> >    3) I create  a dive, then go to Manage, type the first letter of a
> >    dive site. Segmentation fault. On one or two occasions, no crash,
> >    but in > 90% of attempts a crash. It it does not crash:
> 
> Also not perfect.

Ditto.

> >    4) Create a dive, open Manage, fill in dive site name. Blue bar
> >    appears on marble. double-click marble. Seg fault.
> 
> I sense a pattern.

Also fixed

> >    5) I create a dive, open Manage, put cursor on coordinates field.
> >    Now the blue message appears on marble. I double-click marble.
> >    Segmentation fault.
> 
> Yep.

Also fixed

> >    Looks like probably still not possible to add a new dive.
> 
> It was before the latest round of patches. :-)

With latest master this should be possible again. I can see several more
bugs in this code, but I'm running out of time so I pushed what I have and
will try to fix more of this later.

/D