Mercurial > pidgin.yaz
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 |