Re: [GnomeMeeting-devel-list] [PATCH] Sound Input only plugins?
- From: Damien Sandras <dsandras seconix com>
- To: GnomeMeeting development mailing list <gnomemeeting-devel-list gnome org>
- Subject: Re: [GnomeMeeting-devel-list] [PATCH] Sound Input only plugins?
- Date: Tue, 05 Apr 2005 10:09:12 +0200
There is probably still a few problems before going into CVS though.
1) If you use the ALSA plugin, and have a camera with an internal
microphone, the list of input devices will be different from the list of
output devices, so the output plugin will be shown when it shouldn't.
What I suggest is to change that into the following :
- if the output plugin returns zero devices and the input plugin doesn't
return zero devices, then allow to change it.
- if not, then hide it.
Isn't that enough for a test?
2)
I don't understand this :
+ // FIXME: hide/show the output_plugin_menu... there has to be a
better way.
+ gm_conf_set_string (AUDIO_DEVICES_KEY "output_plugin",
+ gm_conf_get_string(AUDIO_DEVICES_KEY "output_plugin"));
3) druid.cpp has a couple of FIXME and some code commented out
The rest seems ok :)
Thanks!
Le mardi 05 avril 2005 à 15:15 +0900, Georgi Georgiev a écrit :
> maillog: 04/04/2005-19:53:18(+0200): Damien Sandras types
> >
> > Excellent idea! If you can implement this, it will go as is in CVS
> > because that is really the best solution.
> >
> > There is no need to present a choice to the user if it doesn't make
> > sense...
>
> Well, here it is.
>
> - The UI logic for the prefs window is added to manager_changed_nt in
> src/config.cpp (I hope that was the right place).
>
> - The druid in this patch is almost the same (GUI-wize) as the original
> one. I quickly had to revert some of my changes, so most of the split
> audio plugins code is there, it is just that the druid sets the output
> to be the same as the input.
>
> - Screenshots: http://gg3.net/~chutz/ss/gm/
> - druid-2.png shows the only noticeable difference in the interface
> (it displays both input and output managers on the last page)
> - prefs-?.png shows the prefs interface (it should probably be made a
> bit nicer, but I didn7t know how to hide the label).
>
> _______________________________________________
> Gnomemeeting-devel-list mailing list
> Gnomemeeting-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/gnomemeeting-devel-list
--
_ Damien Sandras
(o- GnomeMeeting: http://www.gnomemeeting.org/
//\ FOSDEM 2005 : http://www.fosdem.org
v_/_ H.323 phone : callto:ils.seconix.com/dsandras seconix com
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]