[pulseaudio-discuss] [PATCH RFCv3 32/51] resampler: Drop pointless remix variable

David Henningsson david.henningsson at canonical.com
Wed Nov 5 10:35:25 PST 2014


Ack

On 2014-11-05 00:26, Peter Meerwald wrote:
> From: Peter Meerwald <p.meerwald at bct-electronic.com>
>
> Signed-off-by: Peter Meerwald <pmeerw at pmeerw.net>
> ---
>   src/pulsecore/resampler.c | 7 -------
>   1 file changed, 7 deletions(-)
>
> diff --git a/src/pulsecore/resampler.c b/src/pulsecore/resampler.c
> index 5e9dc39..8b30c24 100644
> --- a/src/pulsecore/resampler.c
> +++ b/src/pulsecore/resampler.c
> @@ -737,7 +737,6 @@ static void setup_remap(const pa_resampler *r, pa_remap_t *m) {
>       unsigned oc, ic;
>       unsigned n_oc, n_ic;
>       bool ic_connected[PA_CHANNELS_MAX];
> -    bool remix;
>       pa_strbuf *s;
>       char *t;
>
> @@ -755,16 +754,12 @@ static void setup_remap(const pa_resampler *r, pa_remap_t *m) {
>       memset(m->map_table_i, 0, sizeof(m->map_table_i));
>
>       memset(ic_connected, 0, sizeof(ic_connected));
> -    remix = (r->flags & (PA_RESAMPLER_NO_REMAP | PA_RESAMPLER_NO_REMIX)) == 0;
>
>       if (r->flags & PA_RESAMPLER_NO_REMAP) {
> -        pa_assert(!remix);
> -
>           for (oc = 0; oc < PA_MIN(n_ic, n_oc); oc++)
>               m->map_table_f[oc][oc] = 1.0f;
>
>       } else if (r->flags & PA_RESAMPLER_NO_REMIX) {
> -        pa_assert(!remix);
>           for (oc = 0; oc < n_oc; oc++) {
>               pa_channel_position_t b = r->o_cm.map[oc];
>
> @@ -849,8 +844,6 @@ static void setup_remap(const pa_resampler *r, pa_remap_t *m) {
>               ic_unconnected_lfe = 0;
>           bool ic_unconnected_center_mixed_in = 0;
>
> -        pa_assert(remix);
> -
>           for (ic = 0; ic < n_ic; ic++) {
>               if (on_left(r->i_cm.map[ic]))
>                   ic_left++;
>

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic


More information about the pulseaudio-discuss mailing list