diff options
author | wm4 <wm4@nowhere> | 2013-11-18 14:16:08 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2013-11-18 14:21:01 +0100 |
commit | 5594718b6bda3a230e2e2c3cb06d2837c5a02688 (patch) | |
tree | dca02cab9bde6d50d148a62b20ec99aa8601b847 /audio/filter/af_center.c | |
parent | 93852b08f37f630b994d126751b4b9740a13219f (diff) | |
download | mpv-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.c | 14 |
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; } |