[pulseaudio-discuss] [PATCH 4/4] sink, sink-input: Code cleanup regarding volume ramping

[Sangchul Lee]

Remove unused parameter of
pa_sink_input_set_volume_ramp()/pa_sink_set_volume_ramp().
Use bool instead of pa_bool_t.

Signed-off-by: Sangchul Lee <sc11.lee at samsung.com>
---
 src/pulsecore/sink-input.c | 7 ++-----
 src/pulsecore/sink-input.h | 2 +-
 src/pulsecore/sink.c       | 3 +--
 src/pulsecore/sink.h       | 2 +-
 4 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/pulsecore/sink-input.c b/src/pulsecore/sink-input.c
index 50dc992..7fa9c84 100644
--- a/src/pulsecore/sink-input.c
+++ b/src/pulsecore/sink-input.c
@@ -929,7 +929,6 @@ void pa_sink_input_peek(pa_sink_input *i, size_t
slength /* in sink bytes */, pa
             pa_memchunk wchunk;
             bool nvfs = need_volume_factor_sink;
             pa_cvolume target;
-            pa_bool_t tmp;

             wchunk = tchunk;
             pa_memblock_ref(wchunk.memblock);
@@ -970,7 +969,7 @@ void pa_sink_input_peek(pa_sink_input *i, size_t
slength /* in sink bytes */, pa
                 if (pa_cvolume_ramp_active(&i->thread_info.ramp)) {
                     pa_memchunk_make_writable(&wchunk, 0);
                     pa_volume_ramp_memchunk(&wchunk,
&i->sink->sample_spec, &(i->thread_info.ramp));
-                } else if ((tmp =
pa_cvolume_ramp_target_active(&(i->thread_info.ramp)))) {
+                } else if
((pa_cvolume_ramp_target_active(&(i->thread_info.ramp)))) {
                     pa_memchunk_make_writable(&wchunk, 0);
                     pa_cvolume_ramp_get_targets(&i->thread_info.ramp, &target);
                     pa_volume_memchunk(&wchunk,
&i->sink->sample_spec, &target);
@@ -1370,9 +1369,7 @@ int
pa_sink_input_remove_volume_factor(pa_sink_input *i, const char *key)
{
 void pa_sink_input_set_volume_ramp(
         pa_sink_input *i,
         const pa_cvolume_ramp *ramp,
-        pa_bool_t send_msg,
-        pa_bool_t save) {
-
+        bool send_msg) {
     pa_sink_input_assert_ref(i);
     pa_assert_ctl_context();
     pa_assert(PA_SINK_INPUT_IS_LINKED(i->state));
diff --git a/src/pulsecore/sink-input.h b/src/pulsecore/sink-input.h
index 85a0e54..0efc7e0 100644
--- a/src/pulsecore/sink-input.h
+++ b/src/pulsecore/sink-input.h
@@ -376,7 +376,7 @@ void pa_sink_input_set_volume(pa_sink_input *i,
const pa_cvolume *volume, bool s
 void pa_sink_input_add_volume_factor(pa_sink_input *i, const char
*key, const pa_cvolume *volume_factor);
 int pa_sink_input_remove_volume_factor(pa_sink_input *i, const char *key);
 pa_cvolume *pa_sink_input_get_volume(pa_sink_input *i, pa_cvolume
*volume, bool absolute);
-void pa_sink_input_set_volume_ramp(pa_sink_input *i, const
pa_cvolume_ramp *ramp, pa_bool_t send_msg, pa_bool_t save);
+void pa_sink_input_set_volume_ramp(pa_sink_input *i, const
pa_cvolume_ramp *ramp, bool send_msg);

 void pa_sink_input_set_mute(pa_sink_input *i, bool mute, bool save);

diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
index 0450f52..82b5268 100644
--- a/src/pulsecore/sink.c
+++ b/src/pulsecore/sink.c
@@ -2144,8 +2144,7 @@ void pa_sink_set_volume(
 void pa_sink_set_volume_ramp(
         pa_sink *s,
         const pa_cvolume_ramp *ramp,
-        pa_bool_t send_msg,
-        pa_bool_t save) {
+        bool send_msg) {

     pa_sink_assert_ref(s);
     pa_assert_ctl_context();
diff --git a/src/pulsecore/sink.h b/src/pulsecore/sink.h
index 85f1ddf..d4993af 100644
--- a/src/pulsecore/sink.h
+++ b/src/pulsecore/sink.h
@@ -462,12 +462,12 @@ void pa_sink_leave_passthrough(pa_sink *s);

 void pa_sink_set_volume(pa_sink *sink, const pa_cvolume *volume, bool
sendmsg, bool save);
 const pa_cvolume *pa_sink_get_volume(pa_sink *sink, bool force_refresh);
+void pa_sink_set_volume_ramp(pa_sink *s, const pa_cvolume_ramp *ramp,
bool send_msg);

 void pa_sink_set_mute(pa_sink *sink, bool mute, bool save);
 bool pa_sink_get_mute(pa_sink *sink, bool force_refresh);

 bool pa_sink_update_proplist(pa_sink *s, pa_update_mode_t mode,
pa_proplist *p);
-void pa_sink_set_volume_ramp(pa_sink *s, const pa_cvolume_ramp *ramp,
pa_bool_t send_msg, pa_bool_t save);

 int pa_sink_set_port(pa_sink *s, const char *name, bool save);
 void pa_sink_set_mixer_dirty(pa_sink *s, bool is_dirty);
-- 
2.7.4