Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-23 Thread Michael Niedermayer
On Fri, Feb 23, 2018 at 11:15:50AM -0300, James Almer wrote: > On 2/23/2018 4:35 AM, Hendrik Leppkes wrote: > > On Fri, Feb 23, 2018 at 1:28 AM, James Almer wrote: > >> On 2/22/2018 8:52 PM, Hendrik Leppkes wrote: > >>> On Fri, Feb 23, 2018 at 12:37 AM, James Almer

Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-23 Thread James Almer
On 2/23/2018 4:35 AM, Hendrik Leppkes wrote: > On Fri, Feb 23, 2018 at 1:28 AM, James Almer wrote: >> On 2/22/2018 8:52 PM, Hendrik Leppkes wrote: >>> On Fri, Feb 23, 2018 at 12:37 AM, James Almer wrote: On 2/22/2018 8:34 PM, Hendrik Leppkes wrote:

Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-22 Thread Hendrik Leppkes
On Fri, Feb 23, 2018 at 1:28 AM, James Almer wrote: > On 2/22/2018 8:52 PM, Hendrik Leppkes wrote: >> On Fri, Feb 23, 2018 at 12:37 AM, James Almer wrote: >>> On 2/22/2018 8:34 PM, Hendrik Leppkes wrote: On Fri, Feb 23, 2018 at 12:20 AM, James Almer

Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-22 Thread James Almer
On 2/22/2018 8:52 PM, Hendrik Leppkes wrote: > On Fri, Feb 23, 2018 at 12:37 AM, James Almer wrote: >> On 2/22/2018 8:34 PM, Hendrik Leppkes wrote: >>> On Fri, Feb 23, 2018 at 12:20 AM, James Almer wrote: av_parser_change() is effectively a noop if the

Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-22 Thread Hendrik Leppkes
On Fri, Feb 23, 2018 at 12:37 AM, James Almer wrote: > On 2/22/2018 8:34 PM, Hendrik Leppkes wrote: >> On Fri, Feb 23, 2018 at 12:20 AM, James Almer wrote: >>> av_parser_change() is effectively a noop if the avctx passed it to >>> doesn't have the global

Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-22 Thread James Almer
On 2/22/2018 8:34 PM, Hendrik Leppkes wrote: > On Fri, Feb 23, 2018 at 12:20 AM, James Almer wrote: >> av_parser_change() is effectively a noop if the avctx passed it to >> doesn't have the global header or local header flags set, and >> initializing a custom AVCodecContext as

Re: [FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-22 Thread Hendrik Leppkes
On Fri, Feb 23, 2018 at 12:20 AM, James Almer wrote: > av_parser_change() is effectively a noop if the avctx passed it to > doesn't have the global header or local header flags set, and > initializing a custom AVCodecContext as a copy of an > AVCodecParameters results in the

[FFmpeg-devel] [PATCH] ffmpeg: pass the correct AVCodecContext to av_parser_change()

2018-02-22 Thread James Almer
av_parser_change() is effectively a noop if the avctx passed it to doesn't have the global header or local header flags set, and initializing a custom AVCodecContext as a copy of an AVCodecParameters results in the flags and flags2 fields being zero. Use instead the existing custom