[PATCH] Use gtk_tree_model_get_value() to get the index and date of a dive

[Dirk Hohndel]

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

> On 19 September 2012 02:17, Dirk Hohndel <dirk at hohndel.org> wrote:
>> "Lubomir I. Ivanov" <neolit123 at gmail.com> writes:
>>> well, unfortunately, in "my book" only. :-(
>>> i have some experience with truly broken vararg / cdecl compiler
>>> implementations...
>>
>> Ah. We use so many vararg functions in subsurface... if we have to work
>> around that - oh my. Every printf/scanf function, etc...
>>
>
> yeah,
>
> i think, that is why this is a volatile fix and the real cause might
> be a memory issue. i.e. something writing somewhere, where it
> shouldn't.
> very much possible...and annoyingly hard to debug, especially
> considering the fact that GDB somehow crashes the entire OS (Ubuntu)
> for me.

Yikes. That's encouraging. You might want to submit THAT to Ubuntu for a
fix.

The thing that really annoyed me is that Valgrind is seeing anything out
of the ordinary. Not for the xmlCleanupParser one, nor for this one. If
we are stomping all over memory, isn't Valgrind supposed to catch that?

I pushed all the other fixes. I'm still contemplating this one...

There's also Christian's email about cppcheck to look at. Maybe fixing
those bugs (the ones that actually ARE bugs) will as a side effect fix
this one?

I've started on the list and am looking for the actual bugs that could
mess up our memory...

/D