-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am Mi den  5. Nov 2008 um  8:25 schrieb Laurent MONIN:
> I don't see the point of this patch, code was disabled and a FIXME added 
> because
> it wasn't equivalent to older code.
> Omari Stephens proposed a fix in its message "Two patches: fix segfault, and 
> update deprecated GTK API calls".
> It has to be tested.

Well, the buggy patch did just replace "#if GTK_CHECK_VERSION(2,12,0)"
with "#if 0" so that the first function is never used and only the
second is used. But in GTK+ > 2.12.0 the function
gtk_tree_view_tree_to_widget_coords() is not available but
gtk_tree_view_convert_bin_window_to_widget_coords() works well so with
the "#if 0" there is no way to link together the code as there is no
such function.

Beside of that, the functions might be different but to disable a
compilable version completely is no solution for that.

> Things can be simpler: just use  __FUNCTION__ instead of a different
> message in each function.

Nice. I did search for that. :) How was that with the tree and the
forest!? Well, as I did not insert the debugging calls by hand there was
no problem for me searching for __FUNCTION__ as it takes simply no more
work to add the function name.

> For example, instead of DEBUG_5("Entering function help_window_show()");
> something like DEBUG_F();
> with 
> #define DEBUG_F() DEBUG_5("%s", __FUNCTION__)

I Agree.

Regards
   Klaus Ethgen
- -- 
Klaus Ethgen                            http://www.ethgen.de/
pub  2048R/D1A4EDE5 2000-02-26 Klaus Ethgen <[EMAIL PROTECTED]>
Fingerprint: D7 67 71 C4 99 A6 D4 FE  EA 40 30 57 3C 88 26 2B
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iQEVAwUBSRFUf5+OKpjRpO3lAQLyrQf/bp5yz10JoY3jend5OFqNSnYybmHXEZrL
teEqQMl7ixWAbTtvYVRJ/NOlEaWkysIjV5uYpupTKqb983sLDZKKdffbLYDdwfPz
oPTLiE/p/2ukuwCKC8VwGQULPk+Sw84dgmdJ8uGDGyKjC4X+ZIBwrjRoR2vkNUzM
YKc9YXtyYDJEpxBWn+NuKp7OpGyfHfqtjUC6bdEKKHyIm4BZkpyQ1PmgqQUBiYX7
xvP5ZuCz9XubGOHy6KHlAX3p62gYKIGuJ8gllazYgQV2kqeFtsssA0lpz0uN5/R1
MGPrf8JBBkLFdCUmqH3Vz/g5UozTC5wfbuRolQzmWsNKHhLPaeqWdQ==
=Yums
-----END PGP SIGNATURE-----

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Reply via email to