On 19/10/16 14:24, Hendrik Leppkes wrote:
If its entirely untested (due to being unsupported), it should not be made available, and instead be added once it is supported and has been confirmed to be working properly. There is also no reason to add it if we know it doesn't work, it would just be misleading to add an option, which then shows up in the help output, if we already know it doesnt work anywhere (yet).
No, it's not. It's been tested and working. It allows users to test it's availability in combination with hardware and libraries. It's even working when the capability test says it isn't supported, in which case the encoder continues to produce a valid output stream (this has been tested by me before I implemented the availability test).
What it may not do is to meet your expectations in the output. Yet, temporal aq can in fact produce an identical output in certain cases, which you shouldn't have a problem with. It's also not misleading when in the absence of an availability you then gets an error message informing you about the lack of support.
So I still don't see why you would want to object here. I personally see it as being on the same level with the latest developments by Nvidia.
May I ask, are you strongly against it? Sven _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
