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 |
||
|---|---|---|
| .. | ||
| context.c | ||
| context.h | ||
| d3d11_helpers.c | ||
| d3d11_helpers.h | ||
| error_diffusion.c | ||
| error_diffusion.h | ||
| hwdec.c | ||
| hwdec.h | ||
| lcms.c | ||
| lcms.h | ||
| libmpv_gpu.c | ||
| libmpv_gpu.h | ||
| osd.c | ||
| osd.h | ||
| ra.c | ||
| ra.h | ||
| shader_cache.c | ||
| shader_cache.h | ||
| spirv.c | ||
| spirv.h | ||
| spirv_shaderc.c | ||
| user_shaders.c | ||
| user_shaders.h | ||
| utils.c | ||
| utils.h | ||
| video.c | ||
| video.h | ||
| video_shaders.c | ||
| video_shaders.h | ||