mirror of https://github.com/mpv-player/mpv
419c44ccf6
Internally, vo_gpu uses NaN for some options to indicate a default value that is different depending on the context (e.g. different scalers). There are 2 problems with this: 1. you couldn't reset the options to their defaults 2. NaN is a damn mess and shouldn't be part of the API The option parser already rejected NaN explicitly, which is why 1. didn't work. Regarding 2., JSON might be a good example, and actually caused a bug report. Fix this by mapping NaN to the special value "default". I think I'd prefer other mechanisms (maybe just having every scaler expose separate options?), but for now this will do. See you in a future commit, which painfully deprecates this and replaces it with something else. I refrained from using "no" (my favorite magic value for "unset" etc.) because then I'd have e.g. make --no-scale-param1 work, which in addition to a lot of effort looks dumb and nobody will use it. Here's also an apology for the shitty added test script. Fixes: #6691 |
||
|---|---|---|
| .. | ||
| lua | ||
| mpv-osd-symbols.sfdir | ||
| osxbundle/mpv.app/Contents | ||
| __init__.py | ||
| appveyor-build.sh | ||
| appveyor-install.sh | ||
| dylib-unhell.py | ||
| file2string.py | ||
| gen-osd-font.sh | ||
| idet.sh | ||
| matroska.py | ||
| mpv_identify.sh | ||
| osxbundle.py | ||
| stats-conv.py | ||
| travis-deps | ||
| travis-rebuild-website | ||
| umpv | ||
| uncrustify.cfg | ||