diff options
| author | wm4 <wm4@nowhere> | 2016-09-01 14:21:32 +0200 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2016-09-01 14:21:32 +0200 |
| commit | 6b4f560f3ccca0f7db345bd7a6ed65e9307b432d (patch) | |
| tree | 831199a941a1bf34959ca3809460655eb05f7685 /video | |
| parent | ec3c428e5fb03244b9f8f4f7432779b45a181166 (diff) | |
| download | mpv-6b4f560f3ccca0f7db345bd7a6ed65e9307b432d.tar.bz2 mpv-6b4f560f3ccca0f7db345bd7a6ed65e9307b432d.tar.xz | |
vo, ao: disable positional parameter suboptions
Positional parameters cause problems because they can be ambiguous with
flag options. If a flag option is removed or turned into a non-flag
option, it'll usually be interpreted as value for the first sub-option
(as positional parameter), resulting in very confusing error messages.
This changes it into a simple "option not found" error.
I don't expect that anyone really used positional parameters with --vo
or --ao. Although the docs for --ao=pulse seem to encourage positional
parameters for the host/sink options, which means it could possibly
annoy some PulseAudio users.
--vf and --af are still mostly used with positional parameters, so this
must be a configurable option in the option parser.
Diffstat (limited to 'video')
| -rw-r--r-- | video/out/vo.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/video/out/vo.c b/video/out/vo.c index 31a1e74405..53525b4049 100644 --- a/video/out/vo.c +++ b/video/out/vo.c @@ -193,6 +193,7 @@ const struct m_obj_list vo_obj_list = { }, .allow_unknown_entries = true, .allow_trailer = true, + .disallow_positional_parameters = true, }; static void dispatch_wakeup_cb(void *ptr) |