Re: [GnomeMeeting-list] bug in 0.85.1?

[Damien Sandras <dsandras seconix com>]

On Fri, Jun 07, 2002 at 09:40:53AM -0700, Guilhem Tardy wrote:
> In pref_window.cpp...
> 
> was "/apps/gnomemeeting/devices/video_size", probably a copy&paste error?

Yes, I just verified and it is ok in 0.92 and CVS snapshots.

> 
> BTW, what is the purpose of the following:
>   /* Set the data to be able to block the signal later */
>   gtk_object_set_data (GTK_OBJECT (pw->opt2), "gconf_key",
> 		       (void *) "/apps/gnomemeeting/devices/video_format");
> 
>   gtk_signal_connect (GTK_OBJECT (GTK_OPTION_MENU (pw->opt2)->menu), 
> 		      "deactivate",
>  		      GTK_SIGNAL_FUNC (option_menu_changed), 
> 		      (gpointer) gtk_object_get_data (GTK_OBJECT (pw->opt2),
> 						      "gconf_key"));
> 

It is rather long to explain. But in 2 words, it is to prevent a recursion
of the signal when an external source like gconf-editor changes the value
of the the key.

- a widget changes, it modifies the gconf key, modifying the gconf key, 
takes the appropriate action, and modifies the widget so that it displays
the current gconf_key value (in case you are using gconf-editor or gconftool
to modify the value directly when GM is running). The above thing permits
to block the signal that is emitted when a widget changes so that it doesn't
update the gconf_key again and makes a recursion.


Damien