[FFmpeg-devel] [PATCH 2/3] avcodec/libx264:setting profile and level in avcodec context

Jeyapal, Karthick kjeyapal at akamai.com
Thu Nov 23 14:56:47 EET 2017

On 11/23/17, 5:33 PM, "Carl Eugen Hoyos" <ceffmpeg at gmail.com> wrote:

>2017-11-23 12:47 GMT+01:00 Jeyapal, Karthick <kjeyapal at akamai.com>:
>> On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" <ceffmpeg at gmail.com> wrote:
>>>2017-11-23 4:37 GMT+01:00  <vdixit at akamai.com>:
>>>> +    s = x264_encoder_headers(x4->enc, &nal, &nnal);
>>>> +    if (avctx->profile == FF_PROFILE_UNKNOWN)
>>>> +        avctx->profile = nal->p_payload[5];
>>>> +    if (avctx->level == FF_LEVEL_UNKNOWN)
>>>> +        avctx->level = nal->p_payload[7];
>>>Why are these conditional?
>> We didn’t want to overwrite profile and level, if user had already set it.
>So if x264 changes these values because of contradicting user
>requests, we write the wrong values into the hls header?

Yes, that is true. 
We were afraid to set profile and level unconditionally, because we thought it could get rejected by maintainers(due to that comment in avcodec.h).
Now, if you suggest us to remove those conditions, we would be happy to do it. 
Please advise.

Thanks and regards,

More information about the ffmpeg-devel mailing list