]> code.delx.au - pulseaudio/blobdiff - src/modules/module-equalizer-sink.c
Add pa_sample_rate_valid()
[pulseaudio] / src / modules / module-equalizer-sink.c
index e83a41d540e01268db8c7b2bf1096e175d409346..22043f0733de245079a4a657cd8e7ac21e912310 100644 (file)
@@ -3,7 +3,7 @@
 
   This module is based off Lennart Poettering's LADSPA sink and swaps out
   LADSPA functionality for a dbus-aware STFT OLA based digital equalizer.
-  All new work is published under Pulseaudio's original license.
+  All new work is published under PulseAudio's original license.
 
   Copyright 2009 Jason Newton <nevion@gmail.com>
 
@@ -71,7 +71,7 @@
 PA_MODULE_AUTHOR("Jason Newton");
 PA_MODULE_DESCRIPTION(_("General Purpose Equalizer"));
 PA_MODULE_VERSION(PACKAGE_VERSION);
-PA_MODULE_LOAD_ONCE(FALSE);
+PA_MODULE_LOAD_ONCE(false);
 PA_MODULE_USAGE(
         _("sink_name=<name of the sink> "
           "sink_properties=<properties for the sink> "
@@ -85,13 +85,13 @@ PA_MODULE_USAGE(
          ));
 
 #define MEMBLOCKQ_MAXLENGTH (16*1024*1024)
-#define DEFAULT_AUTOLOADED FALSE
+#define DEFAULT_AUTOLOADED false
 
 struct userdata {
     pa_module *module;
     pa_sink *sink;
     pa_sink_input *sink_input;
-    pa_bool_t autoloaded;
+    bool autoloaded;
 
     size_t channels;
     size_t fft_size;//length (res) of fft
@@ -122,7 +122,7 @@ struct userdata {
     size_t output_buffer_length;
     size_t output_buffer_max_length;
     pa_memblockq *output_q;
-    pa_bool_t first_iteration;
+    bool first_iteration;
 
     pa_dbus_protocol *dbus_protocol;
     char *dbus_path;
@@ -155,19 +155,19 @@ static const char* const valid_modargs[] = {
 static void dbus_init(struct userdata *u);
 static void dbus_done(struct userdata *u);
 
-static void hanning_window(float *W, size_t window_size){
+static void hanning_window(float *W, size_t window_size) {
     /* h=.5*(1-cos(2*pi*j/(window_size+1)), COLA for R=(M+1)/2 */
     for (size_t i = 0; i < window_size; ++i)
         W[i] = (float).5 * (1 - cos(2*M_PI*i / (window_size+1)));
 }
 
-static void fix_filter(float *H, size_t fft_size){
+static void fix_filter(float *H, size_t fft_size) {
     /* divide out the fft gain */
     for (size_t i = 0; i < fft_size / 2 + 1; ++i)
         H[i] /= fft_size;
 }
 
-static void interpolate(float *signal, size_t length, uint32_t *xs, float *ys, size_t n_points){
+static void interpolate(float *samples, size_t length, uint32_t *xs, float *ys, size_t n_points) {
     /* Note that xs must be monotonically increasing! */
     float x_range_lower, x_range_upper, c0;
 
@@ -189,29 +189,29 @@ static void interpolate(float *signal, size_t length, uint32_t *xs, float *ys, s
         c0 = (x-x_range_lower) / (x_range_upper-x_range_lower);
         pa_assert(c0 >= 0 && c0 <= 1.0);
 
-        signal[x] = ((1.0f - c0) * ys[x_range_lower_i] + c0 * ys[x_range_lower_i + 1]);
+        samples[x] = ((1.0f - c0) * ys[x_range_lower_i] + c0 * ys[x_range_lower_i + 1]);
         while(x >= xs[x_range_lower_i + 1])
             x_range_lower_i++;
     }
 
-    signal[length-1] = ys[n_points-1];
+    samples[length-1] = ys[n_points-1];
 }
 
-static pa_bool_t is_monotonic(const uint32_t *xs, size_t length) {
+static bool is_monotonic(const uint32_t *xs, size_t length) {
     pa_assert(xs);
 
     if (length < 2)
-        return TRUE;
+        return true;
 
     for(size_t i = 1; i < length; ++i)
         if (xs[i] <= xs[i-1])
-            return FALSE;
+            return false;
 
-    return TRUE;
+    return true;
 }
 
 /* ensures memory allocated is a multiple of v_size and aligned */
-static void * alloc(size_t x, size_t s){
+static void * alloc(size_t x, size_t s) {
     size_t f;
     float *t;
 
@@ -222,7 +222,7 @@ static void * alloc(size_t x, size_t s){
     return t;
 }
 
-static void alloc_input_buffers(struct userdata *u, size_t min_buffer_length){
+static void alloc_input_buffers(struct userdata *u, size_t min_buffer_length) {
     if (min_buffer_length <= u->input_buffer_max)
         return;
 
@@ -274,7 +274,6 @@ static int sink_process_msg_cb(pa_msgobject *o, int code, void *data, int64_t of
     return pa_sink_process_msg(o, code, data, offset, chunk);
 }
 
-
 /* Called from main context */
 static int sink_set_state_cb(pa_sink *s, pa_sink_state_t state) {
     struct userdata *u;
@@ -302,7 +301,7 @@ static void sink_request_rewind_cb(pa_sink *s) {
         return;
 
     /* Just hand this one over to the master sink */
-    pa_sink_input_request_rewind(u->sink_input, s->thread_info.rewind_nbytes+pa_memblockq_get_length(u->input_q), TRUE, FALSE, FALSE);
+    pa_sink_input_request_rewind(u->sink_input, s->thread_info.rewind_nbytes+pa_memblockq_get_length(u->input_q), true, false, false);
 }
 
 /* Called from I/O thread context */
@@ -333,7 +332,7 @@ static void sink_set_volume_cb(pa_sink *s) {
         !PA_SINK_INPUT_IS_LINKED(pa_sink_input_get_state(u->sink_input)))
         return;
 
-    pa_sink_input_set_volume(u->sink_input, &s->real_volume, s->save_volume, TRUE);
+    pa_sink_input_set_volume(u->sink_input, &s->real_volume, s->save_volume, true);
 }
 
 /* Called from main context */
@@ -362,20 +361,20 @@ static void dsp_logic(
     const float * restrict H,//The freq. magnitude scalers filter
     const float * restrict W,//The windowing function
     fftwf_complex * restrict output_window,//The transformed windowed src
-    struct userdata *u){
+    struct userdata *u) {
 
     //use a linear-phase sliding STFT and overlap-add method (for each channel)
     //window the data
-    for(size_t j = 0; j < u->window_size; ++j){
+    for(size_t j = 0; j < u->window_size; ++j) {
         dst[j] = X * W[j] * src[j];
     }
-    //zero pad the the remaining fft window
+    //zero pad the remaining fft window
     memset(dst + u->window_size, 0, (u->fft_size - u->window_size) * sizeof(float));
     //Processing is done here!
     //do fft
     fftwf_execute_dft_r2c(u->forward_plan, dst, output_window);
     //perform filtering
-    for(size_t j = 0; j < FILTER_SIZE(u); ++j){
+    for(size_t j = 0; j < FILTER_SIZE(u); ++j) {
         u->output_window[j][0] *= H[j];
         u->output_window[j][1] *= H[j];
     }
@@ -384,18 +383,18 @@ static void dsp_logic(
     ////debug: tests overlapping add
     ////and negates ALL PREVIOUS processing
     ////yields a perfect reconstruction if COLA is held
-    //for(size_t j = 0; j < u->window_size; ++j){
+    //for(size_t j = 0; j < u->window_size; ++j) {
     //    u->work_buffer[j] = u->W[j] * u->input[c][j];
     //}
 
     //overlap add and preserve overlap component from this window (linear phase)
-    for(size_t j = 0; j < u->overlap_size; ++j){
+    for(size_t j = 0; j < u->overlap_size; ++j) {
         u->work_buffer[j] += overlap[j];
         overlap[j] = dst[u->R + j];
     }
     ////debug: tests if basic buffering works
     ////shouldn't modify the signal AT ALL (beyond roundoff)
-    //for(size_t j = 0; j < u->window_size;++j){
+    //for(size_t j = 0; j < u->window_size;++j) {
     //    u->work_buffer[j] = u->input[c][j];
     //}
 
@@ -424,14 +423,14 @@ static void dsp_logic(
     const float * restrict H,//The freq. magnitude scalers filter
     const float * restrict W,//The windowing function
     fftwf_complex * restrict output_window,//The transformed windowed src
-    struct userdata *u){//Collection of constants
+    struct userdata *u) {//Collection of constants
     const size_t overlap_size = PA_ROUND_UP(u->overlap_size, v_size);
     float_vector_t x;
     x.f[0] = x.f[1] = x.f[2] = x.f[3] = X;
 
     //assert(u->samples_gathered >= u->R);
     //use a linear-phase sliding STFT and overlap-add method
-    for(size_t j = 0; j < u->window_size; j += v_size){
+    for(size_t j = 0; j < u->window_size; j += v_size) {
         //dst[j] = W[j] * src[j];
         float_vector_t *d = (float_vector_t*) (dst + j);
         float_vector_t *w = (float_vector_t*) (W + j);
@@ -441,14 +440,14 @@ static void dsp_logic(
 //        d->v = x->v * w->v * s->v;
 //#endif
     }
-    //zero pad the the remaining fft window
+    //zero pad the remaining fft window
     memset(dst + u->window_size, 0, (u->fft_size - u->window_size) * sizeof(float));
 
     //Processing is done here!
     //do fft
     fftwf_execute_dft_r2c(u->forward_plan, dst, output_window);
     //perform filtering - purely magnitude based
-    for(size_t j = 0; j < FILTER_SIZE; j += v_size / 2){
+    for(size_t j = 0; j < FILTER_SIZE; j += v_size / 2) {
         //output_window[j][0]*=H[j];
         //output_window[j][1]*=H[j];
         float_vector_t *d = (float_vector_t*)( ((float *) output_window) + 2 * j);
@@ -468,12 +467,12 @@ static void dsp_logic(
     ////debug: tests overlapping add
     ////and negates ALL PREVIOUS processing
     ////yields a perfect reconstruction if COLA is held
-    //for(size_t j = 0; j < u->window_size; ++j){
+    //for(size_t j = 0; j < u->window_size; ++j) {
     //    dst[j] = W[j] * src[j];
     //}
 
     //overlap add and preserve overlap component from this window (linear phase)
-    for(size_t j = 0; j < overlap_size; j += v_size){
+    for(size_t j = 0; j < overlap_size; j += v_size) {
         //dst[j]+=overlap[j];
         //overlap[j]+=dst[j+R];
         float_vector_t *d = (float_vector_t*)(dst + j);
@@ -492,7 +491,7 @@ static void dsp_logic(
 
     ////debug: tests if basic buffering works
     ////shouldn't modify the signal AT ALL (beyond roundoff)
-    //for(size_t j = 0; j < u->window_size; ++j){
+    //for(size_t j = 0; j < u->window_size; ++j) {
     //    dst[j] = src[j];
     //}
 
@@ -503,12 +502,12 @@ static void dsp_logic(
 }
 #endif
 
-static void flatten_to_memblockq(struct userdata *u){
+static void flatten_to_memblockq(struct userdata *u) {
     size_t mbs = pa_mempool_block_size_max(u->sink->core->mempool);
     pa_memchunk tchunk;
     char *dst;
     size_t i = 0;
-    while(i < u->output_buffer_length){
+    while(i < u->output_buffer_length) {
         tchunk.index = 0;
         tchunk.length = PA_MIN((u->output_buffer_length - i), mbs);
         tchunk.memblock = pa_memblock_new(u->sink->core->mempool, tchunk.length);
@@ -522,7 +521,7 @@ static void flatten_to_memblockq(struct userdata *u){
     }
 }
 
-static void process_samples(struct userdata *u){
+static void process_samples(struct userdata *u) {
     size_t fs = pa_frame_size(&(u->sink->sample_spec));
     unsigned a_i;
     float *H, X;
@@ -530,14 +529,14 @@ static void process_samples(struct userdata *u){
     pa_assert(u->samples_gathered >= u->window_size);
     iterations = (u->samples_gathered - u->overlap_size) / u->R;
     //make sure there is enough buffer memory allocated
-    if(iterations * u->R * fs > u->output_buffer_max_length){
+    if (iterations * u->R * fs > u->output_buffer_max_length) {
         u->output_buffer_max_length = iterations * u->R * fs;
         pa_xfree(u->output_buffer);
         u->output_buffer = pa_xmalloc(u->output_buffer_max_length);
     }
     u->output_buffer_length = iterations * u->R * fs;
 
-    for(size_t iter = 0; iter < iterations; ++iter){
+    for(size_t iter = 0; iter < iterations; ++iter) {
         offset = iter * u->R * fs;
         for(size_t c = 0;c < u->channels; c++) {
             a_i = pa_aupdate_read_begin(u->a_H[c]);
@@ -554,28 +553,28 @@ static void process_samples(struct userdata *u){
                 u
             );
             pa_aupdate_read_end(u->a_H[c]);
-            if(u->first_iteration){
+            if (u->first_iteration) {
                 /* The windowing function will make the audio ramped in, as a cheap fix we can
                  * undo the windowing (for non-zero window values)
                  */
-                for(size_t i = 0; i < u->overlap_size; ++i){
+                for(size_t i = 0; i < u->overlap_size; ++i) {
                     u->work_buffer[i] = u->W[i] <= FLT_EPSILON ? u->work_buffer[i] : u->work_buffer[i] / u->W[i];
                 }
             }
             pa_sample_clamp(PA_SAMPLE_FLOAT32NE, (uint8_t *) (((float *)u->output_buffer) + c) + offset, fs, u->work_buffer, sizeof(float), u->R);
         }
-        if(u->first_iteration){
-            u->first_iteration = FALSE;
+        if (u->first_iteration) {
+            u->first_iteration = false;
         }
         u->samples_gathered -= u->R;
     }
     flatten_to_memblockq(u);
 }
 
-static void input_buffer(struct userdata *u, pa_memchunk *in){
+static void input_buffer(struct userdata *u, pa_memchunk *in) {
     size_t fs = pa_frame_size(&(u->sink->sample_spec));
     size_t samples = in->length/fs;
-    float *src = (float*) ((uint8_t*) pa_memblock_acquire(in->memblock) + in->index);
+    float *src = pa_memblock_acquire_chunk(in);
     pa_assert(u->samples_gathered + samples <= u->input_buffer_max);
     for(size_t c = 0; c < u->channels; c++) {
         //buffer with an offset after the overlap from previous
@@ -608,7 +607,7 @@ static int sink_input_pop_cb(pa_sink_input *i, size_t nbytes, pa_memchunk *chunk
 
     fs = pa_frame_size(&(u->sink->sample_spec));
     mbs = pa_mempool_block_size_max(u->sink->core->mempool);
-    if(pa_memblockq_get_length(u->output_q) > 0){
+    if (pa_memblockq_get_length(u->output_q) > 0) {
         //pa_log_debug("qsize is %ld", pa_memblockq_get_length(u->output_q));
         goto END;
     }
@@ -619,7 +618,7 @@ static int sink_input_pop_cb(pa_sink_input *i, size_t nbytes, pa_memchunk *chunk
     //mbs = PA_MAX(mbs, u->R);
     //target_samples = PA_MAX(target_samples, mbs);
     //pa_log_debug("target samples: %ld", target_samples);
-    if(u->first_iteration){
+    if (u->first_iteration) {
         //allocate request_size
         target_samples = PA_MAX(target_samples, u->window_size);
     }else{
@@ -703,7 +702,7 @@ static void sink_input_mute_changed_cb(pa_sink_input *i) {
 }
 
 #if 0
-static void reset_filter(struct userdata *u){
+static void reset_filter(struct userdata *u) {
     size_t fs = pa_frame_size(&u->sink->sample_spec);
     size_t max_request;
 
@@ -712,7 +711,7 @@ static void reset_filter(struct userdata *u){
     for(size_t i = 0; i < u->channels; ++i)
         pa_memzero(u->overlap_accum[i], u->overlap_size * sizeof(float));
 
-    u->first_iteration = TRUE;
+    u->first_iteration = true;
     //set buffer size to max request, no overlap copy
     max_request = PA_ROUND_UP(pa_sink_input_get_max_request(u->sink_input) / fs , u->R);
     max_request = PA_MAX(max_request, u->window_size);
@@ -740,7 +739,7 @@ static void sink_input_process_rewind_cb(pa_sink_input *i, size_t nbytes) {
 
         if (amount > 0) {
             //invalidate the output q
-            pa_memblockq_seek(u->input_q, - (int64_t) amount, PA_SEEK_RELATIVE, TRUE);
+            pa_memblockq_seek(u->input_q, - (int64_t) amount, PA_SEEK_RELATIVE, true);
             pa_log("Resetting filter");
             //reset_filter(u); //this is the "proper" thing to do...
         }
@@ -757,6 +756,8 @@ static void sink_input_update_max_rewind_cb(pa_sink_input *i, size_t nbytes) {
     pa_sink_input_assert_ref(i);
     pa_assert_se(u = i->userdata);
 
+    /* FIXME: Too small max_rewind:
+     * https://bugs.freedesktop.org/show_bug.cgi?id=53709 */
     pa_memblockq_set_maxrewind(u->input_q, nbytes);
     pa_sink_set_max_rewind_within_thread(u->sink, nbytes);
 }
@@ -823,6 +824,9 @@ static void sink_input_attach_cb(pa_sink_input *i) {
     max_request = PA_MAX(max_request, u->window_size);
 
     pa_sink_set_max_request_within_thread(u->sink, max_request * fs);
+
+    /* FIXME: Too small max_rewind:
+     * https://bugs.freedesktop.org/show_bug.cgi?id=53709 */
     pa_sink_set_max_rewind_within_thread(u->sink, pa_sink_input_get_max_rewind(i));
 
     pa_sink_attach_within_thread(u->sink);
@@ -844,10 +848,10 @@ static void sink_input_kill_cb(pa_sink_input *i) {
     pa_sink_input_unref(u->sink_input);
     u->sink_input = NULL;
 
-    pa_sink_unref(u->sink);
-    u->sink = NULL;
+    /* Leave u->sink alone for now, it will be cleaned up on module
+     * unload (and it is needed during unload as well). */
 
-    pa_module_unload_request(u->module, TRUE);
+    pa_module_unload_request(u->module, true);
 }
 
 /* Called from IO thread context */
@@ -862,22 +866,22 @@ static void sink_input_state_change_cb(pa_sink_input *i, pa_sink_input_state_t s
     if (PA_SINK_INPUT_IS_LINKED(state) &&
         i->thread_info.state == PA_SINK_INPUT_INIT) {
         pa_log_debug("Requesting rewind due to state change.");
-        pa_sink_input_request_rewind(i, 0, FALSE, TRUE, TRUE);
+        pa_sink_input_request_rewind(i, 0, false, true, true);
     }
 }
 
-static void pack(char **strs, size_t len, char **packed, size_t *length){
+static void pack(char **strs, size_t len, char **packed, size_t *length) {
     size_t t_len = 0;
     size_t headers = (1+len) * sizeof(uint16_t);
     char *p;
-    for(size_t i = 0; i < len; ++i){
+    for(size_t i = 0; i < len; ++i) {
         t_len += strlen(strs[i]);
     }
     *length = headers + t_len;
     p = *packed = pa_xmalloc0(*length);
     *((uint16_t *) p) = (uint16_t) len;
     p += sizeof(uint16_t);
-    for(size_t i = 0; i < len; ++i){
+    for(size_t i = 0; i < len; ++i) {
         uint16_t l = strlen(strs[i]);
         *((uint16_t *) p) = (uint16_t) l;
         p += sizeof(uint16_t);
@@ -885,13 +889,13 @@ static void pack(char **strs, size_t len, char **packed, size_t *length){
         p += l;
     }
 }
-static void unpack(char *str, size_t length, char ***strs, size_t *len){
+static void unpack(char *str, size_t length, char ***strs, size_t *len) {
     char *p = str;
     *len = *((uint16_t *) p);
     p += sizeof(uint16_t);
     *strs = pa_xnew(char *, *len);
 
-    for(size_t i = 0; i < *len; ++i){
+    for(size_t i = 0; i < *len; ++i) {
         size_t l = *((uint16_t *) p);
         p += sizeof(uint16_t);
         (*strs)[i] = pa_xnew(char, l + 1);
@@ -900,7 +904,7 @@ static void unpack(char *str, size_t length, char ***strs, size_t *len){
         p += l;
     }
 }
-static void save_profile(struct userdata *u, size_t channel, char *name){
+static void save_profile(struct userdata *u, size_t channel, char *name) {
     unsigned a_i;
     const size_t profile_size = CHANNEL_PROFILE_SIZE(u) * sizeof(float);
     float *H_n, *profile;
@@ -911,7 +915,7 @@ static void save_profile(struct userdata *u, size_t channel, char *name){
     profile[0] = u->Xs[a_i][channel];
     H = u->Hs[channel][a_i];
     H_n = profile + 1;
-    for(size_t i = 0 ; i < FILTER_SIZE(u); ++i){
+    for(size_t i = 0 ; i < FILTER_SIZE(u); ++i) {
         H_n[i] = H[i] * u->fft_size;
         //H_n[i] = H[i];
     }
@@ -920,15 +924,15 @@ static void save_profile(struct userdata *u, size_t channel, char *name){
     key.size = strlen(key.data);
     data.data = profile;
     data.size = profile_size;
-    pa_database_set(u->database, &key, &data, TRUE);
+    pa_database_set(u->database, &key, &data, true);
     pa_database_sync(u->database);
-    if(u->base_profiles[channel]){
+    if (u->base_profiles[channel]) {
         pa_xfree(u->base_profiles[channel]);
     }
     u->base_profiles[channel] = pa_xstrdup(name);
 }
 
-static void save_state(struct userdata *u){
+static void save_state(struct userdata *u) {
     unsigned a_i;
     const size_t filter_state_size = FILTER_STATE_SIZE(u) * sizeof(float);
     float *H_n, *state;
@@ -944,7 +948,7 @@ static void save_state(struct userdata *u){
     memcpy(state + FILTER_STATE_SIZE(u), packed, packed_length);
     pa_xfree(packed);
 
-    for(size_t c = 0; c < u->channels; ++c){
+    for(size_t c = 0; c < u->channels; ++c) {
         a_i = pa_aupdate_read_begin(u->a_H[c]);
         state[c * CHANNEL_PROFILE_SIZE(u)] = u->Xs[c][a_i];
         H = u->Hs[c][a_i];
@@ -958,17 +962,17 @@ static void save_state(struct userdata *u){
     data.data = state;
     data.size = filter_state_size + packed_length;
     //thread safety for 0.9.17?
-    pa_assert_se(dbname = pa_state_path(EQ_STATE_DB, FALSE));
-    pa_assert_se(database = pa_database_open(dbname, TRUE));
+    pa_assert_se(dbname = pa_state_path(EQ_STATE_DB, false));
+    pa_assert_se(database = pa_database_open(dbname, true));
     pa_xfree(dbname);
 
-    pa_database_set(database, &key, &data, TRUE);
+    pa_database_set(database, &key, &data, true);
     pa_database_sync(database);
     pa_database_close(database);
     pa_xfree(state);
 }
 
-static void remove_profile(pa_core *c, char *name){
+static void remove_profile(pa_core *c, char *name) {
     pa_datum key;
     pa_database *database;
     key.data = name;
@@ -978,14 +982,14 @@ static void remove_profile(pa_core *c, char *name){
     pa_database_sync(database);
 }
 
-static const char* load_profile(struct userdata *u, size_t channel, char *name){
+static const char* load_profile(struct userdata *u, size_t channel, char *name) {
     unsigned a_i;
     pa_datum key, value;
     const size_t profile_size = CHANNEL_PROFILE_SIZE(u) * sizeof(float);
     key.data = name;
     key.size = strlen(key.data);
-    if(pa_database_get(u->database, &key, &value) != NULL){
-        if(value.size == profile_size){
+    if (pa_database_get(u->database, &key, &value) != NULL) {
+        if (value.size == profile_size) {
             float *profile = (float *) value.data;
             a_i = pa_aupdate_write_begin(u->a_H[channel]);
             u->Xs[channel][a_i] = profile[0];
@@ -1004,16 +1008,16 @@ static const char* load_profile(struct userdata *u, size_t channel, char *name){
     return NULL;
 }
 
-static void load_state(struct userdata *u){
+static void load_state(struct userdata *u) {
     unsigned a_i;
     float *H;
     pa_datum key, value;
     pa_database *database;
     char *dbname;
-    pa_assert_se(dbname = pa_state_path(EQ_STATE_DB, FALSE));
-    database = pa_database_open(dbname, FALSE);
+    pa_assert_se(dbname = pa_state_path(EQ_STATE_DB, false));
+    database = pa_database_open(dbname, false);
     pa_xfree(dbname);
-    if(!database){
+    if (!database) {
         pa_log("No resume state");
         return;
     }
@@ -1021,12 +1025,12 @@ static void load_state(struct userdata *u){
     key.data = u->sink->name;
     key.size = strlen(key.data);
 
-    if(pa_database_get(database, &key, &value) != NULL){
-        if(value.size > FILTER_STATE_SIZE(u) * sizeof(float) + sizeof(uint16_t)){
+    if (pa_database_get(database, &key, &value) != NULL) {
+        if (value.size > FILTER_STATE_SIZE(u) * sizeof(float) + sizeof(uint16_t)) {
             float *state = (float *) value.data;
             size_t n_profs;
             char **names;
-            for(size_t c = 0; c < u->channels; ++c){
+            for(size_t c = 0; c < u->channels; ++c) {
                 a_i = pa_aupdate_write_begin(u->a_H[c]);
                 H = state + c * CHANNEL_PROFILE_SIZE(u) + 1;
                 u->Xs[c][a_i] = state[c * CHANNEL_PROFILE_SIZE(u)];
@@ -1035,7 +1039,7 @@ static void load_state(struct userdata *u){
             }
             unpack(((char *)value.data) + FILTER_STATE_SIZE(u) * sizeof(float), value.size - FILTER_STATE_SIZE(u) * sizeof(float), &names, &n_profs);
             n_profs = PA_MIN(n_profs, u->channels);
-            for(size_t c = 0; c < n_profs; ++c){
+            for(size_t c = 0; c < n_profs; ++c) {
                 pa_xfree(u->base_profiles[c]);
                 u->base_profiles[c] = names[c];
             }
@@ -1049,12 +1053,15 @@ static void load_state(struct userdata *u){
 }
 
 /* Called from main context */
-static pa_bool_t sink_input_may_move_to_cb(pa_sink_input *i, pa_sink *dest) {
+static bool sink_input_may_move_to_cb(pa_sink_input *i, pa_sink *dest) {
     struct userdata *u;
 
     pa_sink_input_assert_ref(i);
     pa_assert_se(u = i->userdata);
 
+    if (u->autoloaded)
+        return false;
+
     return u->sink != dest;
 }
 
@@ -1085,7 +1092,7 @@ int pa__init(pa_module*m) {
     unsigned c;
     float *H;
     unsigned a_i;
-    pa_bool_t use_volume_sharing = TRUE;
+    bool use_volume_sharing = true;
 
     pa_assert(m);
 
@@ -1154,7 +1161,7 @@ int pa__init(pa_module*m) {
     u->inverse_plan = fftwf_plan_dft_c2r_1d(u->fft_size, u->output_window, u->work_buffer, FFTW_ESTIMATE);
 
     hanning_window(u->W, u->window_size);
-    u->first_iteration = TRUE;
+    u->first_iteration = true;
 
     u->base_profiles = pa_xnew0(char *, u->channels);
     for (c = 0; c < u->channels; ++c)
@@ -1203,7 +1210,7 @@ int pa__init(pa_module*m) {
     pa_sink_set_set_mute_callback(u->sink, sink_set_mute_cb);
     if (!use_volume_sharing) {
         pa_sink_set_set_volume_callback(u->sink, sink_set_volume_cb);
-        pa_sink_enable_decibel_volume(u->sink, TRUE);
+        pa_sink_enable_decibel_volume(u->sink, true);
     }
     u->sink->userdata = u;
 
@@ -1220,7 +1227,7 @@ int pa__init(pa_module*m) {
     pa_sink_input_new_data_init(&sink_input_data);
     sink_input_data.driver = __FILE__;
     sink_input_data.module = m;
-    pa_sink_input_new_data_set_sink(&sink_input_data, master, FALSE);
+    pa_sink_input_new_data_set_sink(&sink_input_data, master, false);
     sink_input_data.origin_sink = u->sink;
     pa_proplist_sets(sink_input_data.proplist, PA_PROP_MEDIA_NAME, "Equalized Stream");
     pa_proplist_sets(sink_input_data.proplist, PA_PROP_MEDIA_ROLE, "filter");
@@ -1571,9 +1578,9 @@ static pa_dbus_interface_info equalizer_info={
     .n_signals=EQUALIZER_SIGNAL_MAX
 };
 
-void dbus_init(struct userdata *u){
+void dbus_init(struct userdata *u) {
     uint32_t dummy;
-    DBusMessage *signal = NULL;
+    DBusMessage *message = NULL;
     pa_idxset *sink_list = NULL;
     u->dbus_protocol=pa_dbus_protocol_get(u->sink->core);
     u->dbus_path=pa_sprintf_malloc("/org/pulseaudio/core1/sink%d", u->sink->index);
@@ -1581,12 +1588,12 @@ void dbus_init(struct userdata *u){
     pa_dbus_protocol_add_interface(u->dbus_protocol, u->dbus_path, &equalizer_info, u);
     sink_list = pa_shared_get(u->sink->core, SINKLIST);
     u->database = pa_shared_get(u->sink->core, EQDB);
-    if(sink_list == NULL){
+    if (sink_list == NULL) {
         char *dbname;
         sink_list=pa_idxset_new(&pa_idxset_trivial_hash_func, &pa_idxset_trivial_compare_func);
         pa_shared_set(u->sink->core, SINKLIST, sink_list);
-        pa_assert_se(dbname = pa_state_path("equalizer-presets", FALSE));
-        pa_assert_se(u->database = pa_database_open(dbname, TRUE));
+        pa_assert_se(dbname = pa_state_path("equalizer-presets", false));
+        pa_assert_se(u->database = pa_database_open(dbname, true));
         pa_xfree(dbname);
         pa_shared_set(u->sink->core, EQDB, u->database);
         pa_dbus_protocol_add_interface(u->dbus_protocol, MANAGER_PATH, &manager_info, u->sink->core);
@@ -1594,25 +1601,25 @@ void dbus_init(struct userdata *u){
     }
     pa_idxset_put(sink_list, u, &dummy);
 
-    pa_assert_se((signal = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_SINK_ADDED].name)));
-    dbus_message_append_args(signal, DBUS_TYPE_OBJECT_PATH, &u->dbus_path, DBUS_TYPE_INVALID);
-    pa_dbus_protocol_send_signal(u->dbus_protocol, signal);
-    dbus_message_unref(signal);
+    pa_assert_se((message = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_SINK_ADDED].name)));
+    dbus_message_append_args(message, DBUS_TYPE_OBJECT_PATH, &u->dbus_path, DBUS_TYPE_INVALID);
+    pa_dbus_protocol_send_signal(u->dbus_protocol, message);
+    dbus_message_unref(message);
 }
 
-void dbus_done(struct userdata *u){
+void dbus_done(struct userdata *u) {
     pa_idxset *sink_list;
     uint32_t dummy;
 
-    DBusMessage *signal = NULL;
-    pa_assert_se((signal = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_SINK_REMOVED].name)));
-    dbus_message_append_args(signal, DBUS_TYPE_OBJECT_PATH, &u->dbus_path, DBUS_TYPE_INVALID);
-    pa_dbus_protocol_send_signal(u->dbus_protocol, signal);
-    dbus_message_unref(signal);
+    DBusMessage *message = NULL;
+    pa_assert_se((message = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_SINK_REMOVED].name)));
+    dbus_message_append_args(message, DBUS_TYPE_OBJECT_PATH, &u->dbus_path, DBUS_TYPE_INVALID);
+    pa_dbus_protocol_send_signal(u->dbus_protocol, message);
+    dbus_message_unref(message);
 
     pa_assert_se(sink_list=pa_shared_get(u->sink->core,SINKLIST));
     pa_idxset_remove_by_data(sink_list,u,&dummy);
-    if(pa_idxset_size(sink_list)==0){
+    if (pa_idxset_size(sink_list)==0) {
         pa_dbus_protocol_unregister_extension(u->dbus_protocol, EXTNAME);
         pa_dbus_protocol_remove_interface(u->dbus_protocol, MANAGER_PATH, manager_info.name);
         pa_shared_remove(u->sink->core, EQDB);
@@ -1628,16 +1635,16 @@ void dbus_done(struct userdata *u){
 void manager_handle_remove_profile(DBusConnection *conn, DBusMessage *msg, void *_u) {
     DBusError error;
     pa_core *c = (pa_core *)_u;
-    DBusMessage *signal = NULL;
+    DBusMessage *message = NULL;
     pa_dbus_protocol *dbus_protocol;
     char *name;
     pa_assert(conn);
     pa_assert(msg);
     pa_assert(c);
     dbus_error_init(&error);
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                  DBUS_TYPE_STRING, &name,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
@@ -1645,19 +1652,19 @@ void manager_handle_remove_profile(DBusConnection *conn, DBusMessage *msg, void
     remove_profile(c,name);
     pa_dbus_send_empty_reply(conn, msg);
 
-    pa_assert_se((signal = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_PROFILES_CHANGED].name)));
+    pa_assert_se((message = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_PROFILES_CHANGED].name)));
     dbus_protocol = pa_dbus_protocol_get(c);
-    pa_dbus_protocol_send_signal(dbus_protocol, signal);
+    pa_dbus_protocol_send_signal(dbus_protocol, message);
     pa_dbus_protocol_unref(dbus_protocol);
-    dbus_message_unref(signal);
+    dbus_message_unref(message);
 }
 
-void manager_get_revision(DBusConnection *conn, DBusMessage *msg, void *_u){
+void manager_get_revision(DBusConnection *conn, DBusMessage *msg, void *_u) {
     uint32_t rev=1;
     pa_dbus_send_basic_value_reply(conn, msg, DBUS_TYPE_UINT32, &rev);
 }
 
-static void get_sinks(pa_core *u, char ***names, unsigned *n_sinks){
+static void get_sinks(pa_core *u, char ***names, unsigned *n_sinks) {
     void *iter = NULL;
     struct userdata *sink_u = NULL;
     uint32_t dummy;
@@ -1669,13 +1676,13 @@ static void get_sinks(pa_core *u, char ***names, unsigned *n_sinks){
     pa_assert_se(sink_list = pa_shared_get(u, SINKLIST));
     *n_sinks = (unsigned) pa_idxset_size(sink_list);
     *names = *n_sinks > 0 ? pa_xnew0(char *,*n_sinks) : NULL;
-    for(uint32_t i = 0; i < *n_sinks; ++i){
+    for(uint32_t i = 0; i < *n_sinks; ++i) {
         sink_u = (struct userdata *) pa_idxset_iterate(sink_list, &iter, &dummy);
         (*names)[i] = pa_xstrdup(sink_u->dbus_path);
     }
 }
 
-void manager_get_sinks(DBusConnection *conn, DBusMessage *msg, void *_u){
+void manager_get_sinks(DBusConnection *conn, DBusMessage *msg, void *_u) {
     unsigned n;
     char **names = NULL;
     pa_assert(conn);
@@ -1684,18 +1691,18 @@ void manager_get_sinks(DBusConnection *conn, DBusMessage *msg, void *_u){
 
     get_sinks((pa_core *) _u, &names, &n);
     pa_dbus_send_basic_array_variant_reply(conn, msg, DBUS_TYPE_OBJECT_PATH, names, n);
-    for(unsigned i = 0; i < n; ++i){
+    for(unsigned i = 0; i < n; ++i) {
         pa_xfree(names[i]);
     }
     pa_xfree(names);
 }
 
-static void get_profiles(pa_core *c, char ***names, unsigned *n){
+static void get_profiles(pa_core *c, char ***names, unsigned *n) {
     char *name;
     pa_database *database;
     pa_datum key, next_key;
     pa_strlist *head=NULL, *iter;
-    pa_bool_t done;
+    bool done;
     pa_assert_se(database = pa_shared_get(c, EQDB));
 
     pa_assert(c);
@@ -1703,7 +1710,7 @@ static void get_profiles(pa_core *c, char ***names, unsigned *n){
     pa_assert(n);
     done = !pa_database_first(database, &key, NULL);
     *n = 0;
-    while(!done){
+    while(!done) {
         done = !pa_database_next(database, &key, &next_key, NULL);
         name=pa_xmalloc(key.size + 1);
         memcpy(name, key.data, key.size);
@@ -1716,14 +1723,14 @@ static void get_profiles(pa_core *c, char ***names, unsigned *n){
     }
     (*names) = *n > 0 ? pa_xnew0(char *, *n) : NULL;
     iter=head;
-    for(unsigned i = 0; i < *n; ++i){
+    for(unsigned i = 0; i < *n; ++i) {
         (*names)[*n - 1 - i] = pa_xstrdup(pa_strlist_data(iter));
         iter = pa_strlist_next(iter);
     }
     pa_strlist_free(head);
 }
 
-void manager_get_profiles(DBusConnection *conn, DBusMessage *msg, void *_u){
+void manager_get_profiles(DBusConnection *conn, DBusMessage *msg, void *_u) {
     char **names;
     unsigned n;
     pa_assert(conn);
@@ -1732,13 +1739,13 @@ void manager_get_profiles(DBusConnection *conn, DBusMessage *msg, void *_u){
 
     get_profiles((pa_core *)_u, &names, &n);
     pa_dbus_send_basic_array_variant_reply(conn, msg, DBUS_TYPE_STRING, names, n);
-    for(unsigned i = 0; i < n; ++i){
+    for(unsigned i = 0; i < n; ++i) {
         pa_xfree(names[i]);
     }
     pa_xfree(names);
 }
 
-void manager_get_all(DBusConnection *conn, DBusMessage *msg, void *_u){
+void manager_get_all(DBusConnection *conn, DBusMessage *msg, void *_u) {
     pa_core *c;
     char **names = NULL;
     unsigned n;
@@ -1758,14 +1765,14 @@ void manager_get_all(DBusConnection *conn, DBusMessage *msg, void *_u){
 
     get_sinks(c, &names, &n);
     pa_dbus_append_basic_array_variant_dict_entry(&dict_iter,manager_handlers[MANAGER_HANDLER_EQUALIZED_SINKS].property_name, DBUS_TYPE_OBJECT_PATH, names, n);
-    for(unsigned i = 0; i < n; ++i){
+    for(unsigned i = 0; i < n; ++i) {
         pa_xfree(names[i]);
     }
     pa_xfree(names);
 
     get_profiles(c, &names, &n);
     pa_dbus_append_basic_array_variant_dict_entry(&dict_iter, manager_handlers[MANAGER_HANDLER_PROFILES].property_name, DBUS_TYPE_STRING, names, n);
-    for(unsigned i = 0; i < n; ++i){
+    for(unsigned i = 0; i < n; ++i) {
         pa_xfree(names[i]);
     }
     pa_xfree(names);
@@ -1777,13 +1784,13 @@ void manager_get_all(DBusConnection *conn, DBusMessage *msg, void *_u){
 void equalizer_handle_seed_filter(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u = _u;
     DBusError error;
-    DBusMessage *signal = NULL;
+    DBusMessage *message = NULL;
     float *ys;
     uint32_t *xs, channel, r_channel;
     double *_ys, preamp;
     unsigned x_npoints, y_npoints, a_i;
     float *H;
-    pa_bool_t points_good = TRUE;
+    bool points_good = true;
 
     pa_assert(conn);
     pa_assert(msg);
@@ -1791,43 +1798,43 @@ void equalizer_handle_seed_filter(DBusConnection *conn, DBusMessage *msg, void *
 
     dbus_error_init(&error);
 
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
                 DBUS_TYPE_ARRAY, DBUS_TYPE_UINT32, &xs, &x_npoints,
                 DBUS_TYPE_ARRAY, DBUS_TYPE_DOUBLE, &_ys, &y_npoints,
                 DBUS_TYPE_DOUBLE, &preamp,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
     }
-    for(size_t i = 0; i < x_npoints; ++i){
-        if(xs[i] >= FILTER_SIZE(u)){
-            points_good = FALSE;
+    for(size_t i = 0; i < x_npoints; ++i) {
+        if (xs[i] >= FILTER_SIZE(u)) {
+            points_good = false;
             break;
         }
     }
-    if(!is_monotonic(xs, x_npoints) || !points_good){
+    if (!is_monotonic(xs, x_npoints) || !points_good) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "xs must be monotonic and 0<=x<=%zd", u->fft_size / 2);
         dbus_error_free(&error);
         return;
-    }else if(x_npoints != y_npoints || x_npoints < 2 || x_npoints > FILTER_SIZE(u)){
+    }else if (x_npoints != y_npoints || x_npoints < 2 || x_npoints > FILTER_SIZE(u)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "xs and ys must be the same length and 2<=l<=%zd!", FILTER_SIZE(u));
         dbus_error_free(&error);
         return;
-    }else if(xs[0] != 0 || xs[x_npoints - 1] != u->fft_size / 2){
+    }else if (xs[0] != 0 || xs[x_npoints - 1] != u->fft_size / 2) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "xs[0] must be 0 and xs[-1]=fft_size/2");
         dbus_error_free(&error);
         return;
     }
 
     ys = pa_xmalloc(x_npoints * sizeof(float));
-    for(uint32_t i = 0; i < x_npoints; ++i){
+    for(uint32_t i = 0; i < x_npoints; ++i) {
         ys[i] = (float) _ys[i];
     }
     r_channel = channel == u->channels ? 0 : channel;
@@ -1836,8 +1843,8 @@ void equalizer_handle_seed_filter(DBusConnection *conn, DBusMessage *msg, void *
     u->Xs[r_channel][a_i] = preamp;
     interpolate(H, FILTER_SIZE(u), xs, ys, x_npoints);
     fix_filter(H, u->fft_size);
-    if(channel == u->channels){
-        for(size_t c = 1; c < u->channels; ++c){
+    if (channel == u->channels) {
+        for(size_t c = 1; c < u->channels; ++c) {
             unsigned b_i = pa_aupdate_write_begin(u->a_H[c]);
             float *H_p = u->Hs[c][b_i];
             u->Xs[c][b_i] = preamp;
@@ -1848,12 +1855,11 @@ void equalizer_handle_seed_filter(DBusConnection *conn, DBusMessage *msg, void *
     pa_aupdate_write_end(u->a_H[r_channel]);
     pa_xfree(ys);
 
-
     pa_dbus_send_empty_reply(conn, msg);
 
-    pa_assert_se((signal = dbus_message_new_signal(u->dbus_path, EQUALIZER_IFACE, equalizer_signals[EQUALIZER_SIGNAL_FILTER_CHANGED].name)));
-    pa_dbus_protocol_send_signal(u->dbus_protocol, signal);
-    dbus_message_unref(signal);
+    pa_assert_se((message = dbus_message_new_signal(u->dbus_path, EQUALIZER_IFACE, equalizer_signals[EQUALIZER_SIGNAL_FILTER_CHANGED].name)));
+    pa_dbus_protocol_send_signal(u->dbus_protocol, message);
+    dbus_message_unref(message);
 }
 
 void equalizer_handle_get_filter_points(DBusConnection *conn, DBusMessage *msg, void *_u) {
@@ -1862,7 +1868,7 @@ void equalizer_handle_get_filter_points(DBusConnection *conn, DBusMessage *msg,
     double *ys, preamp;
     unsigned x_npoints, a_i;
     float *H;
-    pa_bool_t points_good=TRUE;
+    bool points_good=true;
     DBusMessage *reply = NULL;
     DBusMessageIter msg_iter;
     DBusError error;
@@ -1872,28 +1878,28 @@ void equalizer_handle_get_filter_points(DBusConnection *conn, DBusMessage *msg,
     pa_assert(u);
 
     dbus_error_init(&error);
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
                 DBUS_TYPE_ARRAY, DBUS_TYPE_UINT32, &xs, &x_npoints,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
     }
 
-    for(size_t i = 0; i < x_npoints; ++i){
-        if(xs[i] >= FILTER_SIZE(u)){
-            points_good=FALSE;
+    for(size_t i = 0; i < x_npoints; ++i) {
+        if (xs[i] >= FILTER_SIZE(u)) {
+            points_good=false;
             break;
         }
     }
 
-    if(x_npoints > FILTER_SIZE(u) || !points_good){
+    if (x_npoints > FILTER_SIZE(u) || !points_good) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "xs indices/length must be <= %zd!", FILTER_SIZE(u));
         dbus_error_free(&error);
         return;
@@ -1904,7 +1910,7 @@ void equalizer_handle_get_filter_points(DBusConnection *conn, DBusMessage *msg,
     a_i = pa_aupdate_read_begin(u->a_H[r_channel]);
     H = u->Hs[r_channel][a_i];
     preamp = u->Xs[r_channel][a_i];
-    for(uint32_t i = 0; i < x_npoints; ++i){
+    for(uint32_t i = 0; i < x_npoints; ++i) {
         ys[i] = H[xs[i]] * u->fft_size;
     }
     pa_aupdate_read_end(u->a_H[r_channel]);
@@ -1920,14 +1926,14 @@ void equalizer_handle_get_filter_points(DBusConnection *conn, DBusMessage *msg,
     pa_xfree(ys);
 }
 
-static void get_filter(struct userdata *u, size_t channel, double **H_, double *preamp){
+static void get_filter(struct userdata *u, size_t channel, double **H_, double *preamp) {
     float *H;
     unsigned a_i;
     size_t r_channel = channel == u->channels ? 0 : channel;
     *H_ = pa_xnew0(double, FILTER_SIZE(u));
     a_i = pa_aupdate_read_begin(u->a_H[r_channel]);
     H = u->Hs[r_channel][a_i];
-    for(size_t i = 0;i < FILTER_SIZE(u); ++i){
+    for(size_t i = 0;i < FILTER_SIZE(u); ++i) {
         (*H_)[i] = H[i] * u->fft_size;
     }
     *preamp = u->Xs[r_channel][a_i];
@@ -1935,7 +1941,7 @@ static void get_filter(struct userdata *u, size_t channel, double **H_, double *
     pa_aupdate_read_end(u->a_H[r_channel]);
 }
 
-void equalizer_handle_get_filter(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_handle_get_filter(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     unsigned n_coefs;
     uint32_t channel;
@@ -1948,14 +1954,14 @@ void equalizer_handle_get_filter(DBusConnection *conn, DBusMessage *msg, void *_
     pa_assert(msg);
 
     dbus_error_init(&error);
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
@@ -1976,7 +1982,7 @@ void equalizer_handle_get_filter(DBusConnection *conn, DBusMessage *msg, void *_
     pa_xfree(H_);
 }
 
-static void set_filter(struct userdata *u, size_t channel, double *H_, double preamp){
+static void set_filter(struct userdata *u, size_t channel, double *H_, double preamp) {
     unsigned a_i;
     size_t r_channel = channel == u->channels ? 0 : channel;
     float *H;
@@ -1984,12 +1990,12 @@ static void set_filter(struct userdata *u, size_t channel, double *H_, double pr
     a_i = pa_aupdate_write_begin(u->a_H[r_channel]);
     u->Xs[r_channel][a_i] = (float) preamp;
     H = u->Hs[r_channel][a_i];
-    for(size_t i = 0; i < FILTER_SIZE(u); ++i){
+    for(size_t i = 0; i < FILTER_SIZE(u); ++i) {
         H[i] = (float) H_[i];
     }
     fix_filter(H, u->fft_size);
-    if(channel == u->channels){
-        for(size_t c = 1; c < u->channels; ++c){
+    if (channel == u->channels) {
+        for(size_t c = 1; c < u->channels; ++c) {
             unsigned b_i = pa_aupdate_write_begin(u->a_H[c]);
             u->Xs[c][b_i] = u->Xs[r_channel][a_i];
             memcpy(u->Hs[c][b_i], u->Hs[r_channel][a_i], FILTER_SIZE(u) * sizeof(float));
@@ -1999,33 +2005,33 @@ static void set_filter(struct userdata *u, size_t channel, double *H_, double pr
     pa_aupdate_write_end(u->a_H[r_channel]);
 }
 
-void equalizer_handle_set_filter(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_handle_set_filter(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     double *H, preamp;
     uint32_t channel;
     unsigned _n_coefs;
-    DBusMessage *signal = NULL;
+    DBusMessage *message = NULL;
     DBusError error;
     pa_assert_se(u = (struct userdata *) _u);
     pa_assert(conn);
     pa_assert(msg);
 
     dbus_error_init(&error);
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
                 DBUS_TYPE_ARRAY, DBUS_TYPE_DOUBLE, &H, &_n_coefs,
                 DBUS_TYPE_DOUBLE, &preamp,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
     }
-    if(_n_coefs != FILTER_SIZE(u)){
+    if (_n_coefs != FILTER_SIZE(u)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "This filter takes exactly %zd coefficients, you gave %d", FILTER_SIZE(u), _n_coefs);
         return;
     }
@@ -2033,31 +2039,31 @@ void equalizer_handle_set_filter(DBusConnection *conn, DBusMessage *msg, void *_
 
     pa_dbus_send_empty_reply(conn, msg);
 
-    pa_assert_se((signal = dbus_message_new_signal(u->dbus_path, EQUALIZER_IFACE, equalizer_signals[EQUALIZER_SIGNAL_FILTER_CHANGED].name)));
-    pa_dbus_protocol_send_signal(u->dbus_protocol, signal);
-    dbus_message_unref(signal);
+    pa_assert_se((message = dbus_message_new_signal(u->dbus_path, EQUALIZER_IFACE, equalizer_signals[EQUALIZER_SIGNAL_FILTER_CHANGED].name)));
+    pa_dbus_protocol_send_signal(u->dbus_protocol, message);
+    dbus_message_unref(message);
 }
 
 void equalizer_handle_save_profile(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u = (struct userdata *) _u;
     char *name;
     uint32_t channel, r_channel;
-    DBusMessage *signal = NULL;
+    DBusMessage *message = NULL;
     DBusError error;
     pa_assert(conn);
     pa_assert(msg);
     pa_assert(u);
     dbus_error_init(&error);
 
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
                 DBUS_TYPE_STRING, &name,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
@@ -2066,9 +2072,9 @@ void equalizer_handle_save_profile(DBusConnection *conn, DBusMessage *msg, void
     save_profile(u, r_channel, name);
     pa_dbus_send_empty_reply(conn, msg);
 
-    pa_assert_se((signal = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_PROFILES_CHANGED].name)));
-    pa_dbus_protocol_send_signal(u->dbus_protocol, signal);
-    dbus_message_unref(signal);
+    pa_assert_se((message = dbus_message_new_signal(MANAGER_PATH, MANAGER_IFACE, manager_signals[MANAGER_SIGNAL_PROFILES_CHANGED].name)));
+    pa_dbus_protocol_send_signal(u->dbus_protocol, message);
+    dbus_message_unref(message);
 }
 
 void equalizer_handle_load_profile(DBusConnection *conn, DBusMessage *msg, void *_u) {
@@ -2077,22 +2083,22 @@ void equalizer_handle_load_profile(DBusConnection *conn, DBusMessage *msg, void
     DBusError error;
     uint32_t channel, r_channel;
     const char *err_msg = NULL;
-    DBusMessage *signal = NULL;
+    DBusMessage *message = NULL;
 
     pa_assert(conn);
     pa_assert(msg);
     pa_assert(u);
     dbus_error_init(&error);
 
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
                 DBUS_TYPE_STRING, &name,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
@@ -2100,21 +2106,21 @@ void equalizer_handle_load_profile(DBusConnection *conn, DBusMessage *msg, void
     r_channel = channel == u->channels ? 0 : channel;
 
     err_msg = load_profile(u, r_channel, name);
-    if(err_msg != NULL){
+    if (err_msg != NULL) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_FAILED, "error loading profile %s: %s", name, err_msg);
         dbus_error_free(&error);
         return;
     }
-    if(channel == u->channels){
-        for(uint32_t c = 1; c < u->channels; ++c){
+    if (channel == u->channels) {
+        for(uint32_t c = 1; c < u->channels; ++c) {
             load_profile(u, c, name);
         }
     }
     pa_dbus_send_empty_reply(conn, msg);
 
-    pa_assert_se((signal = dbus_message_new_signal(u->dbus_path, EQUALIZER_IFACE, equalizer_signals[EQUALIZER_SIGNAL_FILTER_CHANGED].name)));
-    pa_dbus_protocol_send_signal(u->dbus_protocol, signal);
-    dbus_message_unref(signal);
+    pa_assert_se((message = dbus_message_new_signal(u->dbus_path, EQUALIZER_IFACE, equalizer_signals[EQUALIZER_SIGNAL_FILTER_CHANGED].name)));
+    pa_dbus_protocol_send_signal(u->dbus_protocol, message);
+    dbus_message_unref(message);
 }
 
 void equalizer_handle_save_state(DBusConnection *conn, DBusMessage *msg, void *_u) {
@@ -2127,7 +2133,7 @@ void equalizer_handle_save_state(DBusConnection *conn, DBusMessage *msg, void *_
     pa_dbus_send_empty_reply(conn, msg);
 }
 
-void equalizer_handle_get_profile_name(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_handle_get_profile_name(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u = (struct userdata *) _u;
     DBusError error;
     uint32_t channel, r_channel;
@@ -2137,14 +2143,14 @@ void equalizer_handle_get_profile_name(DBusConnection *conn, DBusMessage *msg, v
     pa_assert(u);
     dbus_error_init(&error);
 
-    if(!dbus_message_get_args(msg, &error,
+    if (!dbus_message_get_args(msg, &error,
                 DBUS_TYPE_UINT32, &channel,
-                DBUS_TYPE_INVALID)){
+                DBUS_TYPE_INVALID)) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "%s", error.message);
         dbus_error_free(&error);
         return;
     }
-    if(channel > u->channels){
+    if (channel > u->channels) {
         pa_dbus_send_error(conn, msg, DBUS_ERROR_INVALID_ARGS, "invalid channel: %d", channel);
         dbus_error_free(&error);
         return;
@@ -2154,12 +2160,12 @@ void equalizer_handle_get_profile_name(DBusConnection *conn, DBusMessage *msg, v
     pa_dbus_send_basic_value_reply(conn,msg, DBUS_TYPE_STRING, &u->base_profiles[r_channel]);
 }
 
-void equalizer_get_revision(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_get_revision(DBusConnection *conn, DBusMessage *msg, void *_u) {
     uint32_t rev=1;
     pa_dbus_send_basic_value_reply(conn, msg, DBUS_TYPE_UINT32, &rev);
 }
 
-void equalizer_get_n_channels(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_get_n_channels(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     uint32_t channels;
     pa_assert_se(u = (struct userdata *) _u);
@@ -2170,7 +2176,7 @@ void equalizer_get_n_channels(DBusConnection *conn, DBusMessage *msg, void *_u){
     pa_dbus_send_basic_variant_reply(conn, msg, DBUS_TYPE_UINT32, &channels);
 }
 
-void equalizer_get_n_coefs(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_get_n_coefs(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     uint32_t n_coefs;
     pa_assert_se(u = (struct userdata *) _u);
@@ -2181,7 +2187,7 @@ void equalizer_get_n_coefs(DBusConnection *conn, DBusMessage *msg, void *_u){
     pa_dbus_send_basic_variant_reply(conn, msg, DBUS_TYPE_UINT32, &n_coefs);
 }
 
-void equalizer_get_sample_rate(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_get_sample_rate(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     uint32_t rate;
     pa_assert_se(u = (struct userdata *) _u);
@@ -2192,7 +2198,7 @@ void equalizer_get_sample_rate(DBusConnection *conn, DBusMessage *msg, void *_u)
     pa_dbus_send_basic_variant_reply(conn, msg, DBUS_TYPE_UINT32, &rate);
 }
 
-void equalizer_get_filter_rate(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_get_filter_rate(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     uint32_t fft_size;
     pa_assert_se(u = (struct userdata *) _u);
@@ -2203,7 +2209,7 @@ void equalizer_get_filter_rate(DBusConnection *conn, DBusMessage *msg, void *_u)
     pa_dbus_send_basic_variant_reply(conn, msg, DBUS_TYPE_UINT32, &fft_size);
 }
 
-void equalizer_get_all(DBusConnection *conn, DBusMessage *msg, void *_u){
+void equalizer_get_all(DBusConnection *conn, DBusMessage *msg, void *_u) {
     struct userdata *u;
     DBusMessage *reply = NULL;
     DBusMessageIter msg_iter, dict_iter;