diff options
| author | wm4 <wm4@nowhere> | 2015-09-10 14:15:12 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2015-09-10 14:15:48 +0200 |
| commit | f1205293a70b13f53bfce0a1a9859132b8421443 (patch) | |
| tree | fb6ead7e0fbfedbfe9454144ef5349f87de60529 /input/cmd_list.c | |
| parent | 2492b5f1198161a04b046554df79390eb312ca5c (diff) | |
| download | mpv-f1205293a70b13f53bfce0a1a9859132b8421443.tar.bz2 mpv-f1205293a70b13f53bfce0a1a9859132b8421443.tar.xz | |
command: make "add <property> 0" not change the value
The value 0 was treated specially, and effectively forced the increment
to 1. Interestingly, passing 0 or no value also does not include the
scale (from touchpads etc.), but this is probably an accidental behavior
that was never intentionally added.
Simplify it and make the default increment 1. 0 now means what it
should: the value will not be changed. This is not particularly useful,
but on the other hand there is no need for surprising and unintuitive
semantics.
OARG_CYCLEDIR() failed to apply the default value, because
m_option_type_cycle_dir was missing a copy handler - add this too.
Diffstat (limited to 'input/cmd_list.c')
| -rw-r--r-- | input/cmd_list.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/input/cmd_list.c b/input/cmd_list.c index 3640cce460..2f3bfb994a 100644 --- a/input/cmd_list.c +++ b/input/cmd_list.c @@ -146,7 +146,7 @@ const struct mp_cmd_def mp_cmds[] = { { MP_CMD_RUN, "run", { ARG_STRING, ARG_STRING }, .vararg = true }, { MP_CMD_SET, "set", { ARG_STRING, ARG_STRING } }, - { MP_CMD_ADD, "add", { ARG_STRING, OARG_DOUBLE(0) }, + { MP_CMD_ADD, "add", { ARG_STRING, OARG_DOUBLE(1) }, .allow_auto_repeat = true}, { MP_CMD_CYCLE, "cycle", { ARG_STRING, |