[FFmpeg-devel] [PATCH 05/11] ffserver: allow skip setting defaults

Lukasz Marek lukasz.m.luki2 at gmail.com
Sat Nov 22 01:13:09 CET 2014


On 21.11.2014 16:31, Reynaldo H. Verdejo Pinochet wrote:
> Hi
>
> On 11/20/2014 10:09 PM, Lukasz Marek wrote:
>> On 18.11.2014 23:25, Reynaldo H. Verdejo Pinochet wrote:
>>> [..]
>>> I do think undefined behavior should be avoided if possible
>>> without too much hassle, so if we go with the former I would
>>> appreciate doc entries specifying the options precedence. Brownie
>>> points+ if an odd combination fires a warning().
>>
>> Not sure I follow, should I change anything? IMHO my proposal it is not
>> really complex and fully customizable.
>
> After Nicolas observation I think it's OK for you to keep
> the original implementation. Just see if you can document
> the options' precedence when an XYES and XNO are present in
> the config file and consider adding a warning in such case.
>
> Thanks a lot!
>
> Bests,
>
updated patch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-ffserver-allow-skip-setting-defaults.patch
Type: text/x-patch
Size: 6244 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141122/1224cdf1/attachment.bin>


More information about the ffmpeg-devel mailing list