Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: stop using AVStream.codec on stream copy

2016-09-27 Thread James Almer
On 9/26/2016 10:28 PM, Michael Niedermayer wrote: > On Mon, Sep 26, 2016 at 08:06:47PM -0300, James Almer wrote: >> This commit is based on commit 35c8580 from Anton Khirnov >> which was skipped in b8945c4. >> >> The avcodec_copy_context() call in the encode path is left in place for now >> as AVS

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: stop using AVStream.codec on stream copy

2016-09-26 Thread Michael Niedermayer
On Mon, Sep 26, 2016 at 08:06:47PM -0300, James Almer wrote: > This commit is based on commit 35c8580 from Anton Khirnov > which was skipped in b8945c4. > > The avcodec_copy_context() call in the encode path is left in place for now > as AVStream.codec is apparently still required even after port

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: stop using AVStream.codec on stream copy

2016-09-26 Thread Michael Niedermayer
On Mon, Sep 26, 2016 at 07:17:20PM -0300, James Almer wrote: > On 9/26/2016 6:38 PM, Michael Niedermayer wrote: > > On Mon, Sep 26, 2016 at 02:39:13PM -0300, James Almer wrote: > >> This commit is based on commit 35c8580 from Anton Khirnov > >> > >> which was skipped in b8945c4. > >> > >> The avc

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: stop using AVStream.codec on stream copy

2016-09-26 Thread James Almer
On 9/26/2016 6:38 PM, Michael Niedermayer wrote: > On Mon, Sep 26, 2016 at 02:39:13PM -0300, James Almer wrote: >> This commit is based on commit 35c8580 from Anton Khirnov >> which was skipped in b8945c4. >> >> The avcodec_copy_context() call in the encode path is left in place for now >> as AVSt

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: stop using AVStream.codec on stream copy

2016-09-26 Thread Michael Niedermayer
On Mon, Sep 26, 2016 at 02:39:13PM -0300, James Almer wrote: > This commit is based on commit 35c8580 from Anton Khirnov > which was skipped in b8945c4. > > The avcodec_copy_context() call in the encode path is left in place for now > as AVStream.codec is apparently still required even after port