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

2017-11-23 Thread Jeyapal, Karthick
On 11/23/17, 8:34 PM, "Carl Eugen Hoyos" wrote: >2017-11-23 13:56 GMT+01:00 Jeyapal, Karthick : >> >> On 11/23/17, 5:33 PM, "Carl Eugen Hoyos" wrote: >> >>>2017-11-23 12:47 GMT+01:00 Jeyapal, Karthick : On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" wrote: >2017-11-23 4:37 GMT+01:00

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

2017-11-23 Thread Carl Eugen Hoyos
2017-11-23 13:56 GMT+01:00 Jeyapal, Karthick : > > On 11/23/17, 5:33 PM, "Carl Eugen Hoyos" wrote: > >>2017-11-23 12:47 GMT+01:00 Jeyapal, Karthick : >>> On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" wrote: >>> 2017-11-23 4:37 GMT+01:00 : >>> > +s = x264_encoder_headers(x4->enc, &nal, &nn

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

2017-11-23 Thread Jeyapal, Karthick
On 11/23/17, 5:33 PM, "Carl Eugen Hoyos" wrote: >2017-11-23 12:47 GMT+01:00 Jeyapal, Karthick : >> On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" wrote: >> >>>2017-11-23 4:37 GMT+01:00 : >> +s = x264_encoder_headers(x4->enc, &nal, &nnal); +if (avctx->profile == FF_PROFILE_UNKNOWN)

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

2017-11-23 Thread Carl Eugen Hoyos
2017-11-23 12:47 GMT+01:00 Jeyapal, Karthick : > On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" wrote: > >>2017-11-23 4:37 GMT+01:00 : >> >>> +s = x264_encoder_headers(x4->enc, &nal, &nnal); >>> +if (avctx->profile == FF_PROFILE_UNKNOWN) >>> +avctx->profile = nal->p_payload[5]; >>> +

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

2017-11-23 Thread Jeyapal, Karthick
On 11/23/17, 4:21 PM, "Carl Eugen Hoyos" wrote: >2017-11-23 4:37 GMT+01:00 : > >> +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

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

2017-11-23 Thread Carl Eugen Hoyos
2017-11-23 4:37 GMT+01:00 : > +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?

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

2017-11-22 Thread vdixit
From: Vishwanath Dixit Signed-off-by: Karthick J --- libavcodec/libx264.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 9c67c91..eeafe31 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -454,6