comparison libpurple/mediamanager.c @ 31805:017b7ff5a894

During a voice call, Pidgin now sends constant audio traffic, even when there is silence. Especially on slower connections, this can waste considerable amount of bandwidth by transmitting nothing but ambient noise. I used peak level data from GstLevel? in the input branch of media pipeline to control a GstValve? put between audio source and Farsight confbin. Whenever the peak drops below defined threshold, the valve gets closed, when sound level reaches above the threshold, valve opens again. This effectively blocks sending data over network in the silent periods and in my tests this simple method worked quite well. Silence threshold might need to be fine tuned (or switched off at all) depending on microphone hardware and/or the noisiness of surrounding environment. I will propose an user interface for this in a separate ticket. Future improvement can be adding support for comfort noise (RFC3389), as the line now stays completely mute when suppression is active, which can be a bit distracting. I made a tiny change in level parameter that is passed to PurpleMedia?'s "level" signal handlers. The value converted from dB to percent was multiplied by five. Searching through source code history seems this was done to make the value variation displayed on call dialog level meter widgets look bigger. I think it is better not to confuse future developers and pass the unmodified percent value to the handler and multiply only in gtkmedia.c: level_message_cb() where it has reason. committer: John Bailey <rekkanoryo@rekkanoryo.org>
author jakub.adam@ktknet.cz
date Sun, 13 Mar 2011 18:00:58 +0000
parents 66fe4bda9a85
children d72d728226dc
comparison
equal deleted inserted replaced
31804:0362da30c62a 31805:017b7ff5a894
173 media->priv->backend_type = PURPLE_TYPE_MEDIA_BACKEND_FS2; 173 media->priv->backend_type = PURPLE_TYPE_MEDIA_BACKEND_FS2;
174 #endif 174 #endif
175 175
176 purple_prefs_add_none("/purple/media"); 176 purple_prefs_add_none("/purple/media");
177 purple_prefs_add_none("/purple/media/audio"); 177 purple_prefs_add_none("/purple/media/audio");
178 purple_prefs_add_int("/purple/media/audio/silence_threshold", 5);
178 purple_prefs_add_none("/purple/media/audio/volume"); 179 purple_prefs_add_none("/purple/media/audio/volume");
179 purple_prefs_add_int("/purple/media/audio/volume/input", 10); 180 purple_prefs_add_int("/purple/media/audio/volume/input", 10);
180 purple_prefs_add_int("/purple/media/audio/volume/output", 10); 181 purple_prefs_add_int("/purple/media/audio/volume/output", 10);
181 } 182 }
182 183