On Wed, 2015-12-16 at 09:09 +0530, arun at accosted.net wrote: > From: Arun Raghavan <git at arunraghavan.net> > > --- > Â src/modules/echo-cancel/webrtc.cc | 6 +++--- > Â 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/modules/echo-cancel/webrtc.cc b/src/modules/echo-cancel/webrtc.cc > index 3be7fe5..a8b69b5 100644 > --- a/src/modules/echo-cancel/webrtc.cc > +++ b/src/modules/echo-cancel/webrtc.cc > @@ -78,9 +78,9 @@ static int routing_mode_from_string(const char *rmode) { > Â Â Â Â Â Â Â Â Â return -1; > Â } > Â > -void pa_webrtc_ec_fixate_spec(pa_sample_spec *rec_ss, pa_channel_map *rec_map, > -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_sample_spec *play_ss, pa_channel_map *play_map, > -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_sample_spec *out_ss, pa_channel_map *out_map) > +static void pa_webrtc_ec_fixate_spec(pa_sample_spec *rec_ss, pa_channel_map *rec_map, > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_sample_spec *play_ss, pa_channel_map *play_map, > +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_sample_spec *out_ss, pa_channel_map *out_map) > Â { > Â Â Â Â Â rec_ss->format = PA_SAMPLE_S16NE; > Â Â Â Â Â play_ss->format = PA_SAMPLE_S16NE; The pa_ prefix should be dropped for static functions, and "webrtc" (maybe also "ec") seems redundant too. --Â Tanu