summaryrefslogtreecommitdiffstats
path: root/audio/filter/af_center.c
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2013-11-18 14:16:08 +0100
committerwm4 <wm4@nowhere>2013-11-18 14:21:01 +0100
commit5594718b6bda3a230e2e2c3cb06d2837c5a02688 (patch)
treedca02cab9bde6d50d148a62b20ec99aa8601b847 /audio/filter/af_center.c
parent93852b08f37f630b994d126751b4b9740a13219f (diff)
downloadmpv-5594718b6bda3a230e2e2c3cb06d2837c5a02688.tar.bz2
mpv-5594718b6bda3a230e2e2c3cb06d2837c5a02688.tar.xz
audio/filter: remove unneeded AF_CONTROLs, convert to enum
The AF control commands used an elaborate and unnecessary organization for the command constants. Get rid of all that and convert the definitions to a simple enum. Also remove the control commands that were not really needed, because they were not used outside of the filters that implemented them.
Diffstat (limited to 'audio/filter/af_center.c')
-rw-r--r--audio/filter/af_center.c14
1 files changed, 4 insertions, 10 deletions
diff --git a/audio/filter/af_center.c b/audio/filter/af_center.c
index b64d5b54bd..fb026350a5 100644
--- a/audio/filter/af_center.c
+++ b/audio/filter/af_center.c
@@ -57,21 +57,15 @@ static int control(struct af_instance* af, int cmd, void* arg)
case AF_CONTROL_COMMAND_LINE:{
int ch=1;
sscanf(arg,"%i", &ch);
- return control(af,AF_CONTROL_CENTER_CH | AF_CONTROL_SET, &ch);
- }
- case AF_CONTROL_CENTER_CH | AF_CONTROL_SET: // Requires reinit
- // Sanity check
- if((*(int*)arg >= AF_NCH) || (*(int*)arg < 0)){
+ if((ch >= AF_NCH) || (ch < 0)){
mp_msg(MSGT_AFILTER, MSGL_ERR, "[sub] Center channel number must be between "
- " 0 and %i current value is %i\n", AF_NCH-1, *(int*)arg);
+ " 0 and %i current value is %i\n", AF_NCH-1, ch);
return AF_ERROR;
}
- s->ch = *(int*)arg;
- return AF_OK;
- case AF_CONTROL_CENTER_CH | AF_CONTROL_GET:
- *(int*)arg = s->ch;
+ s->ch = ch;
return AF_OK;
}
+ }
return AF_UNKNOWN;
}