On 11/23/17, 5:33 PM, "Carl Eugen Hoyos" <[email protected]> wrote:
>2017-11-23 12:47 GMT+01:00 Jeyapal, Karthick <[email protected]>: >> On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" <[email protected]> wrote: >> >>>2017-11-23 4:37 GMT+01:00 <[email protected]>: >> >>>> + 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, Karthick _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
