[PATCH] Select all dives in a trip when said trip is expanded and clicked

Dirk Hohndel dirk at hohndel.org
Mon Feb 18 10:35:30 PST 2013


"Lubomir I. Ivanov" <neolit123 at gmail.com> writes:

> From: "Lubomir I. Ivanov" <neolit123 at gmail.com>
>
> This patch selects visually all dives within a trip when a
> trip is expanded (using gtk_tree_selection_select_iter()).
> If the trip is not expanded it only selects the dives
> as data (using select_dive()).
>
> When calling get_iter_from_idx(), make sure we free memory at
> the returned address.

Yes, I think this is an improvement over the current situation. 
I have not been able to create a situation where this creates a
regression.

It still has one flaw that isn't addressed:

Expand a trip.
Select a dive in the trip
Collapse the trip - all dives in the trip are selected
Expand the trip - all dives are selected

This is not a new bug that you introduced, Lubomir, this is a
shortcoming of Linus' implementation.

I just wanted to point out that that one is still not fixed.

/D


More information about the subsurface mailing list