]> code.delx.au - pulseaudio/commitdiff
echo-cancel: Mark immutable parameters as const in vfunc
authorArun Raghavan <arun.raghavan@collabora.co.uk>
Tue, 7 Sep 2010 09:37:39 +0000 (15:07 +0530)
committerArun Raghavan <arun.raghavan@collabora.co.uk>
Tue, 7 Sep 2010 09:42:12 +0000 (15:12 +0530)
Marks the recording and playback streams as const in the
pa_echo_canceller->run method for clarity.

src/modules/echo-cancel/adrian.c
src/modules/echo-cancel/echo-cancel.h
src/modules/echo-cancel/speex.c

index 1b91373765730d39bf909276697038030bef17f6..86db1e2cb414ea066da5890f13d26b92ee9ca53f 100644 (file)
@@ -93,7 +93,7 @@ fail:
     return FALSE;
 }
 
-void pa_adrian_ec_run(pa_echo_canceller *ec, uint8_t *rec, uint8_t *play, uint8_t *out)
+void pa_adrian_ec_run(pa_echo_canceller *ec, const uint8_t *rec, const uint8_t *play, uint8_t *out)
 {
     unsigned int i;
 
index bf81b1de0282928f4a5fd4b6e422a175adb5929d..448ad994a6f2281d4809248870c85bff75539c36 100644 (file)
@@ -57,7 +57,7 @@ struct pa_echo_canceller {
                                          pa_channel_map *sink_map,
                                          uint32_t *blocksize,
                                          const char *args);
-    void        (*run)                  (pa_echo_canceller *ec, uint8_t *rec, uint8_t *play, uint8_t *out);
+    void        (*run)                  (pa_echo_canceller *ec, const uint8_t *rec, const uint8_t *play, uint8_t *out);
     void        (*done)                 (pa_echo_canceller *ec);
 
     pa_echo_canceller_params params;
@@ -68,7 +68,7 @@ pa_bool_t pa_speex_ec_init(pa_echo_canceller *ec,
                            pa_sample_spec *source_ss, pa_channel_map *source_map,
                            pa_sample_spec *sink_ss, pa_channel_map *sink_map,
                            uint32_t *blocksize, const char *args);
-void pa_speex_ec_run(pa_echo_canceller *ec, uint8_t *rec, uint8_t *play, uint8_t *out);
+void pa_speex_ec_run(pa_echo_canceller *ec, const uint8_t *rec, const uint8_t *play, uint8_t *out);
 void pa_speex_ec_done(pa_echo_canceller *ec);
 
 /* Adrian Andre's echo canceller */
@@ -76,5 +76,5 @@ pa_bool_t pa_adrian_ec_init(pa_echo_canceller *ec,
                            pa_sample_spec *source_ss, pa_channel_map *source_map,
                            pa_sample_spec *sink_ss, pa_channel_map *sink_map,
                            uint32_t *blocksize, const char *args);
-void pa_adrian_ec_run(pa_echo_canceller *ec, uint8_t *rec, uint8_t *play, uint8_t *out);
+void pa_adrian_ec_run(pa_echo_canceller *ec, const uint8_t *rec, const uint8_t *play, uint8_t *out);
 void pa_adrian_ec_done(pa_echo_canceller *ec);
index 0d4d123611702c40179db455858465744b336fbe..17a89d23e33debd2e51369c838c39ed72281640e 100644 (file)
@@ -104,7 +104,7 @@ fail:
     return FALSE;
 }
 
-void pa_speex_ec_run(pa_echo_canceller *ec, uint8_t *rec, uint8_t *play, uint8_t *out)
+void pa_speex_ec_run(pa_echo_canceller *ec, const uint8_t *rec, const uint8_t *play, uint8_t *out)
 {
     speex_echo_cancellation(ec->params.priv.speex.state, (const spx_int16_t *) rec, (const spx_int16_t *) play, (spx_int16_t *) out);
 }